The only way to compile is use ${TARGET_SYS}-gcc --sysroot=${STAGING_DIR_TARGET} ${TARGET_CC_ARCH}. If I don't use --sysroot=${STAGING_DIR_TARGET} the compiler can't find C headers. ${TARGET_CC_ARCH} is to set correct CFLAGS, without this variable build fails if error "gnu/stubs-soft.h: No such file or directory"


混乱を抱かせるようなソースを残してしまい、申し訳ありません。

まず、件の`process.emit`の使い方については、完全に私の間違いです。
(動かないモノを消し忘れでcommitしてしまっていました)

実現したかったこととしては、gulpのprocessからElectronを立ち上げてprocess間通信を行うことで、gulp側からElectronを制御する、というものでした(ユースケース：ファイル変更に応じたBrowserProcess, RenderProcessの再起動等)。

結局、Electronのプロセスの立ち上げ方上、child_processによるプロセス間通信は難しいようで、別途webSocket等のソケットを独自に立ち上げた上で、gulp - Electron間でSocket通信させる方法で実装を進めている所です。
(82ba1a036ce22b1aef3a053e0ceafac7cdcad86b にて修正のコミットを行っています)

また何かありましたら、ご意見を頂けると幸いです


test comment
fix this here.


I think our modification time in stub is based on folders API -- it has to be translated in the fs.js instead.


reformat from folders to what stats expects.


Mobile view: logo is centered and search bar is centered in the row below the logo.
![center logo on small screen](https://cloud.githubusercontent.com/assets/3407469/8041595/582e7582-0de7-11e5-9697-931e0c62b0f3.png)

Normal view: logo stays in the left of the page and search bar is on the same row to the right.
![logo in left position on normal screen](https://cloud.githubusercontent.com/assets/3407469/8041600/64e1abaa-0de7-11e5-8c59-92a80c11f1e9.png)


@cpennington There are pair of queries like these:
```
 'query': [call(0, u'test_xmodule.modulestore_68d47.active_versions', 0, -1, {'org': 'org.45', 'run': 'An_Introduction_to_API_Testing', 'course': 'course_45'}, None, 3),
    call(0, u'test_xmodule.modulestore_68d47.structures', 0, -1, {'_id': ObjectId('5575d78a56c02c1dfa8c0e7d')}, None, 3),
    call(0, u'test_xmodule.modulestore_68d47.active_versions', 0, -1, {'org': 'org.45', 'run': 'An_Introduction_to_API_Testing', 'course': 'course_45'}, None, 3),
    call(0, u'test_xmodule.modulestore_68d47.structures', 0, -1, {'_id': ObjectId('5575d78a56c02c1dfa8c0e7d')}, None, 3),
    call(0, u'test_xmodule.modulestore_68d47.active_versions', 0, -1, {'org': 'org.45', 'run': 'An_Introduction_to_API_Testing', 'course': 'course_45'}, None, 3),
    call(0, u'test_xmodule.modulestore_68d47.structures', 0, -1, {'_id': ObjectId('5575d78a56c02c1dfa8c0e7d')}, None, 3),
    call(0, u'test_xmodule.modulestore_68d47.active_versions', 0, -1, {'org': 'org.45', 'run': 'An_Introduction_to_API_Testing', 'course': 'course_45'}, None, 3),
    call(0, u'test_xmodule.modulestore_68d47.structures', 0, -1, {'_id': ObjectId('5575d78a56c02c1dfa8c0e7d')}, None, 3)]}
```

I'll also add assertions for old mongo as well, but in case of it, in step 3 there is only 1 query per ancestor instead of 2 with split.


This should go in the install button state method


As soon as it worked its fine, we'll merge later


In order for this to work on Windows, I think you need to use `File.separator` or another path concatenation technique.


@pauldowman looks good :necktie:


I like the idea, it is something I've discussed with lucas in the past, to expose APIs like this.


Looks fine.


thanks @junrao for confirming. I'm looking for a working java producer/consumer example. where can I find ?


:sparkles:


никак не отделяется от предыдущего вывода


Will this work if you run the console from any directory?  I would think you need to find the location of the javadocs within a JAR.  Maybe something like this:

http://stackoverflow.com/questions/3309022/java-loading-resources-from-the-file-system


It does not work so ... Why * current_command is equal to 'N' and skip while...


Fixed #1403


Try to cache this, I don't know if we should use setTimeout or what, but readdir is followed by stat loop on each file in many cases -- with our backend that would mean running ls a lot of times unnecessarily.


What is the alternative? Suggests to use Ultimate Edition?


@Sunilkumarr see the left hand side code of AlertDialog, this got removed in your code commit, please add the alert dialog code again and then commit and push.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.250](http://teamcity.monogame.net/viewLog.html?buildId=8144&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Когда task['return_code'] == 'none', получаем ошибку:

Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/tornado/web.py", line 1346, in _execute
    result = method(*self.path_args, **self.path_kwargs)
  File "server.py", line 150, in get
    self.write('  with return code %d <br>' % self.tasks[key]['return_code'])
TypeError: %d format: a number is required, not str


Bug: When switching device off and on again, it will always start up with screen #3. Before 0010, the last screen selected was shown.


Humm, no.

The dbus spec specifically says an initial 0 byte should be sent that has the user creds attached via SCM_CREDS, and that that should be used for authentication. I am pretty sure we should not deviate unnecessarily from that:

http://dbus.freedesktop.org/doc/dbus-specification.html#auth-nul-byte

Moreover, I am pretty sure that if we use the SCM_CREDS info from that NUL byte we should also use the SCM_SECURITY data from it.

It appears that the kernel is quite broken if it sends rubbish SCM_SECURITY for stream sockets. Either it should send valid SCM_SECURITY data, or it should not send it, but certainly no rubbish data. And if it does, that's something to fix in the kernel, and not in systemd.


Не понятно, что такое 'none', лучше использовать None


Use Util.TAG?


Well, I understand your suggestion, but I'm inclined to keep it for 3 reasons:

* Users familiar with the original UI will be expecting to use kubectl proxy. They can still do that, so it seems reasonable to show them how, while pointing them to the README for the new UI, where they can learn about additional ways to access it.
* Many users will come to this page before looking at the UI README, since this page is part of the docs. It seems reasonable to give them instructions on how to use the UI here.
* Note that `/ui` is not the actual address of the UI. It merely redirects to /static/app, which has always been the path to the UI, which has always been built into the api server. Users will therefore see `http://localhost:8080/static/app/` most of the time. It seems reasonable to fully disclose to them that this is the address of the application, and to note that `/ui` is a redirect.


Bug: Battery will not charge more than 87%


The commit message says `...avocado.test.core...` but it's `avocado.core.test`. The commit change is OK. But there's a open point here, sphinx will not put the documentation for `avocado.test.Test`, it will output documentation for `avocado.core.test.Test`.


> Do you like the idea of keeping the prop name defaultSelected?

Yeah, it seems fine to keep this as is, I think, though it is a bit less intuitive now.

> getInitialState function explicitly creates a new array

But we're not comparing state.selected (the copy) to newProps.defaultSelected, we're comparing props.defaultSelected to newProps.defaultSelected.  Regardless, the performance hit will be pretty small for small selections (which should be the common case)


It appears this was missing the `move_file` if the resource didn't exist yet. RE: #29.


New "Power-off-delay" is much nicer.


Now it seems fine.


The java consumer is still being built in trunk. For java producer, you can find some an example in https://github.com/confluentinc/schema-registry/blob/master/core/src/main/java/io/confluent/kafka/schemaregistry/storage/KafkaStore.java#L258


If you don't replace the onSubmit hook, every instance of the form you've visited will submit when you submit the latest copy since they all respond to the same event.

The [other autoform wizard](https://github.com/forwarder/meteor-wizard/blob/master/wizard.js#L187) also replaces the onSubmit hook.


This get_salt() check was what was added.  The code movement of valid, and call of ssl_init() (which was created), was needed to get the code to link and run properly.  But we were coring if valid passed, but then later get_salt() returned a null.


You're absolutely right about the commit message. I did not get the point about the documentation though. The plan is to, in later work, document both the core API and the user accessible API in different sections. Obviously they're the same implementation, but they'd be available in both namespaces.


I'll fix it.


Instead of using a homepage example, we should use an example that has a url similar to a piece of content. Just to reinforce that this is for content tracking. I find that when examples deviate from the philosophy/focus of the tool, I get confused. Like, "they said this was only to track content, but the example is for a homepage, so maybe i should put this on my homepage too."


When you set a location for a campus, it automatically creates a root-level named location.  All the building/rooms should then be created under that location for this to work (but I don't think this is new). Essentially, the room locations just need an ancestor location that has same id as the location associated with the campus.


If you add in a call to `with modulestore().bulk_transaction()`, then it should collapse those queries.


I want our code to be consistent. We are doing using upper case in any other place in qserv like that. If we decide to obey the 3.3, then we should bite the bullet and change it *consistently* through qserv code. For now, I still feel it would be better to use normal camelCase starting from small letter.


For same reasons as in earlier patch: if we expose a per-connection selinux label, i should probably the SCM_SECURITY one from the initial NUL byte.


Something like: "The tutorial works with IntelliJ IDEA Community Edition as well as IntelliJ IDEA Ultimate. For best experience developing Kotlin/JavaScript applications, we recommend using IntelliJ IDEA Ultimate, which includes the JavaScript debugger and other JavaScript support features."


License headers )


I wonder if @ktlim has an opinion on that..


Given that `docs/ui.md` already explains how the app should be accessed, it seems reasonable for this document to be oriented toward developers. In its current location, this section follows immediately from the section above it, which explains how to build the app for production. I'd therefore prefer to leave it where it is.


On Mon, Jun 8, 2015 at 4:19 AM, David Disseldorp <notifications@github.com>
wrote:

Since I did the change that allows multipath tests also for /dev/* nodes,
we need some extra checks in here.
We can only run this particular tests IFF we have an iSCSI connection.

So you probably want to start with a check that all multipath components
are in fact iSCSI.
Something like this :

for(reset_path = 0; reset_path < mp_num_sds; reset_path++) {
  struct scsi_device *reset_sd = mp_sds[reset_path];
  if (reset_sd->iscsi_ctx == NULL) {
     ...not an iscsi device, skipping test...
  }
}

The LU is checked for UAs prior to the reset via a TUR prior to the reset
> call, which I guess would also fail for targets with queued UAs.
> Should I change both calls to iscsi_testunitready_sync(...) loops?
>

Yes, that is probably the best.
In the first place, replace the calls:
+ ret = testunitready(reset_sd, EXPECT_STATUS_GOOD);+ CU_ASSERT_EQUAL(ret, 0);

with a loop that uses iscsi_testunitready_sync()  and loop until you get
status GOOD.
That way you ensure that there are no pending UAs from prior to this test.

And in the second place:

+ ret = testunitready(mp_sds[tur_path],+ SCSI_STATUS_CHECK_CONDITION,+
SCSI_SENSE_UNIT_ATTENTION,+ lu_reset_acsq, sizeof(lu_reset_acsq));
replace this too with a loop over iscsi_testunitready_sync() and check the
UAs until you get status GOOD.

The for this is that I think on some targets you may get several different
UAs after a reset and there is no guarantee that the first one
will be bus reset. I have seen some targets that generate 3-5 different UAs
when you issue a reset.

Also, I don't quite understand "iterate over it until all async events have
> been consumed.". Do you mean loop until the reset UA has been flagged and
> cleared?
>

Yes.

> Thanks for the feedback Ronnie.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ddiss/libiscsi/commit/d839b403173739a91f81e663621c7b17a0c8082a#commitcomment-11565058>
> .
>


Looks good to me.


Ok, I'll fix it.


Не там делаете chmod, надо на файле, а не директории task_path. В результате потом будет ошибка "Permission denied". И вместо "stat.S_IXUSR" (только флаг executable), лучше "0o744" (сохраняет флаги на чтение и запись).


drop the this and make the first two sentences one sentence "...you will need to include the canonical url in this field, which is the permanent (canonical) url of the content." Also I think you can combine the next two sentences as well "This is the url that a user would enter in a browser to visit the page directly and should not include the protocol unless it changes the content being rendered."

These changes also alleviate the need to start three sentences with 'this'. ;)


For the current status of the way we generate the documentation, the things that are imported will not create a proper documentation. So `avocado.test.Test` will contain documentation ,but `avocado.Test` will not.


@joflashstudios we are now!


Operator `>>` shift number to the right, but into left it doesnt fill zeros, but first bit of number - sign of number. Theoreticaly should help using of `>>>` operator, but it was also unsuccessful when I test it.


First sentence needs to be more similar to Short - see the other commands but this is generally a header, no period.


There's a missing point here: `./docs/source/DebuggingWithGDB.rst: from avocado import test`.


borrado


Whoops, I pushed the version where I just commented out the offending code. Proper fix coming with next push.


I think you switched the comments for these...


Looks good, I don't want to block because of documentation.


Yeah! I saw that.
is Google using it as default for all Nexus devices?
Well, I will check these dex2oat values for 2GB and 3GB for sure.
Anyway, thanks for warning me.


@kshefchek I just did this by a dumb clone of the disease page. If you later decide to refactor how we do this no worries, should be easy to change this page


Ugh, nevermind, realized this is in the _other_ package card view code...


@jackgr As you noted in your previous comment, many people will arrive here before looking at `docs/ui.md`. That is why putting "how to access it in production" at the top made more sense to me, then following with developer based instructions.


The intention is to move this plug-in upstream, correct?

If so, shouldn't we put it in Godeps?  under plugin/pkg/admission/securitycontext/constraint?


@ruda right. The point is that the documentation review PR will include a full check of those. I was kind of saving energy instead of replicating things in both PRs.


> we're comparing props.defaultSelected to newProps.defaultSelected

Good point. I see what you're saying now. I don't think react creates copies of the props internally. I have not dug through the source enough to answer definitively.  Thanks for your support.


Technical language here! A list of examples would be much better. Like

In our home directory `~`, we have a directory xxx, and in that directory we have yyy.

Then make a practicum where people refer to a file xxx/yyy , but when they change to directory xxx, they can just say yyy.


Since I've fixed the commit message though, I'll fix the GDB doc too and will include those in a v2.


Shouldn't this be in /Godeps/.../kubernetes/cmd/kube-apiserver/app/plugins.go?


rax retry tests


Yes, sorry for that.


Вы возвращаете не валидный JSON (dict), а строку.


People don't like to eval like this, some strange thing can't happen if you pass a bash command direcly( rm ... etc)

Maybe change it to a test like "~ = $HOME" ?


You already have SOMC_CFG_SENSORS_LIGHT_LIBALS for  libals, why did you copy the same and changed to SOMC_CFG_SENSORS_LIGHT_AS3677_LIBALS ?
Any specific reason?


This is an openshift specific plugin and relies on openshift specific uid/mcs label allocation constructs.


Why do you need to catch the exceptions in onfulfill/onreject?


This seems like a super-important comment; should it be [duplicated here](https://github.com/code-dot-org/code-dot-org/commit/574ae3d69c41d688db60f88766e7b69104fbcb8d#diff-b1946af8f0edaba7311f5dce6a119c95R88)?


It's not a huge deal. A template with `T const&` would be my first choice `std::string const&` or `char const*` are OK too.


oh, so this is what prevents it from going in Godeps?  is there a plan to move this upstream?


OK.


This could be done with some of that fine bourbon magic right @mathijs727?


Did this work ? Or did it require more changes ?


It has to do with when the formula field values are calculated. They are not available in the after phase of the trigger the rollup works in.


Is this preferable to `href="#"`?  I don't know if we use the anchor for anything else.


Не выдается статус для других состояний


This is only needed if libs are in device tree and we don't want the common ones. Also let's be reasonable, we aren't sharing this repos with any device so this isn't need for here.


Hello Rjwalls, i a PhD student from University Tun Hussein Onn Malaysia woking on Mobile forensic

can you please help me with the exact header and footer for Call Log and Address Book to enable me proceed with my codes. thank you.


I'm not sure what the "course-v1" piece represents?


whats poppin


Yes.  @smarterclayton ^


Heh, they don't know `$(...)`


@teg im not sure i can move $(LIBIDN_LIBS) without breaking the build in this commit, since now anything that links to libsystemd-shared.la also needs libidn.


I'm not sure why this example is necessary. The `Game` above isn't an undefined factory, it's more like an `afterCreate` function. Unless I misunderstand it's purpose.


the function is passing $storage


@dianakhuang Would you mind taking a quick look to see whether this makes sense? I modified the course info based on Will's comment on the wiki page. Thanks!


was this change intentional?


yo whats good g


ay homes


This seems like a bit of a kludge. Perhaps the transaction should be set to batched when it's added to the batch and cleared only when it's executed.


line 18: autocomplete_data_static = {};
that's not a good idea, because it will clear all data for all inputs.

I include this in new version 0.1.1.

Thanks for your fix!


I like the idea of this class.  I'd suggest to create an ancestor for slave-side source checkout commands and provide a slightly better error message `slave-side source checkout for '%s' is no longer supported by build slave of version %s`


k


ur


Perfect.


Please change this line.


not


morrison


Please change this line. It doesn't look proper.


Wist niet dat je bourbon had toegevoegd, heb het aangepast.

2015-06-08 20:29 GMT+02:00 Tristan van Vaalen <notifications@github.com>:

> This could be done with some of that fine bourbon magic right @mathijs727
> <https://github.com/mathijs727>?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ArnoldSwaggernegger/ariana/commit/14cda287312286fe9869a64e6044e430a2c1d5b1#commitcomment-11573111>
> .
>


There is a mistake in the version. It should be 9.4-1201-jdbc41_1-SNAPSHOT. The last `_1` identify our build attempt.


@Joimer It doesn't appear that was ever done.

@Slayer95 How would Gen 2 freeze break then? You're not supposed to defrost if you miss Flame Wheel/Sacred Fire, which is why I used `AfterMoveSecondarySelf` for it.


use log_full(some_weird_condition ? LOG_ERROR : LOG_DEBUG, ... ) for cases like this.

or in other words: log_full(LOG_DEBUG, ...) is equivalent to log_debug(...), and log_full(LOG_ERROR, ...) to log_error()...


I cringe a little at the inline markup here but I don't know if I have a better suggestion; I assume jQuery handles iframes just fine?  Maybe:
```javascript
$('<iframe>')
    .addClass('markdown-instructions-container')
    .width('100%')
    .attr('src', this.getDropletTooltip(functionName).getFullDocumentationURL()
```


OK.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.250](http://teamcity.monogame.net/viewLog.html?buildId=8144&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:21:51


Great!


The accept header can be multi value that might be a case we would want to handle.

http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html.


this is not the directory name


Very awkward placement of the front controller. You already have a controllers directory in your lib, so make use of it. It's okay if the filename is called *Front.php*; You'll know it is a controller because of where it is.


If we do this, the seek is now redundant, right?


If we do this, the set_metadata and the logger.debug could be unindented so they are outside of the with block.


Is the bug really that I used set_contents_from_filename instead of set_contents_from_file which caused the duplicate open?

Would it fix the bug on Windows if we left the seek(0) and changed the  key.set_contents_from_filename(tf.name) to  key.set_contents_from_file(tf)?


done.


Hmm... yeah okay.  Worth extracting that string?  I assume this is true across all browsers?


can this be a function?  its copied/pasted a lot, just want a function that gets a namespace if the suppled namespace pointer is nil.

```
namespace, err := findNamespace(namespaceObject *kapi.Namespace, namespace string)
```


While this comment is true, that's not how you are actually using it here... it does something different when you pass in that second parameter.


finally approved.


Sure


Any reason why you pass this in, instead of using events to allow external subscribers to make the choice themselves?


@teg re line 276, result of `dns_label_undo_idna` is k, the length. this is passed to `dns_label_escape`. it could probably be changed to just pass r to `dns_label_escape`.


Really? `me.Vector2d` already has this: https://github.com/melonjs/melonJS/blob/2b2fc9a63add8596581e6123529fb7a509a501ba/src/math/vector2.js#L22


I'm not sure I understand, it is used.


That seems like it should be `course-v1:OrganizationX+Coursex.2+CourseRun` (no beginning slash). The `course-v1` represents the type of the course id it is. Generally people won't need to know what it means.


Do you really expect to have multiple controller directories per module? This is very non-standard and seems like overkill. Especially because you are requiring 4 pieces in each URL as opposed to the typical 3 pieces (module, controller, action).


Use the overload that allows seconds - I think sub minute intervals will be common


My opinion is that this particular const string is unusable outside its function and is only used in one place in its function, which in turn is only two lines.  Just make it a string constant in the `LOG_GET`.

As to what we should be doing in the rest of the Qserv code, we need to have some level of consistency across all LSST code, not just Qserv code.  If you can show that there are fewer ALL_CAPS constants in the rest of the stack than mixedCase constants in Qserv, we can talk about changing the standards.


What was the doc comment removed? If it's no longer useful it should be tagged `@ignore` or similar, don't you think?


no, the instance won't crash or should not crash, just the error servlet will be called and print the cause. Hmmh, but still maybe you are right and we should catch it.


Are you sure you don't want to use `this._super` ? Especially since the `set` method returns a reference to the vector...


I wonder if having them grab the actual course id from the course would be clearer and make more sense? The format of their course ids might be the old slash-separated type, so these instructions might not work if they try to construct the course id by hand.


@SamTay this is his params array. It could be important, but he could also get these from the request object.


Because pos is already declared in the parent constructor?

> On 9 juin 2015, at 02:41, Jay Oster <notifications@github.com> wrote:
>
> What was the doc comment removed? If it's no longer useful it should be tagged @ignore or similar, don't you think?
>
> —
> Reply to this email directly or view it on GitHub.
>


This actually worked on my local Haxe version which was built from OCaml 3.11. It fails when built with OCaml 4.


Also, the `--insecure` flag only should be used for sandboxes. Both here and on L159, I don't think we should include it.


I think we should start them on the `$(...)` syntax right off the bat. It's more sensible.


Might want to rename this - you're not returning a schedule. Looks like you're returning a delta between next occurrence and now?


Yep, I tried actually, but I ended up doing too many different things...


@jackgr Let me clarify a few thoughts:
- I was only referring to the _Running Locally_ section as the one which should be removed. It's confusing to have different instructions when the web ui is now baked into the apiserver, and that is the recommended way of accessing it. If anyone wants to run it locally in a modified capacity, they can be referred to `www/README.md` for development instructions. Aside from that case, I don't see a point in running another instance of the same ui with the `proxy` command.
- I agree that users will see `/static/app` most of the time, and I'm fine with disclosing that `/ui` is a redirect. `/ui` is much friendlier to remember though.


das gefaellt mir nich. es sollte ein hotplug-script sein. nehmt mal dieses:
https://github.com/bittorf/kalua/blob/master/openwrt-addons/etc/hotplug.d/iface/50-olsrd


Do we really need a deep copy of the previous pod?  Or can we just created a kapi.Pod{} and set a few fields?  I need to step through what provider.CreateSecurityContext and provider/ValidateSecurityContext is doing to really require a full copy of a pod versus just select fields, but this seems hackish


I just wanted to point out that the documentation inside the code block uses a .map(function() { this.route... and here it is much more concise. Is there a reason why the code was changed?


Could have negative impact on performance. Maybe we should better convert to correct types on instantiation.


@ruda any comments about this one? If not, I'll send a v2. Thanks.


```
$this->container->make
```
returns a FatalErrorException: Call to a member function make() on null
When tests break code…


That's a really good idea. I'll tell them how to get the course ID from the course URL.


If you expect to keep controllers in a single directory (called controller or controllers), then you would never need to pass ``$controllerDir`` through the URL. Just set the standard and use it the same way every time:
    $className = $module . '_controller_' . $controllerFile
And in that case, don't use ``$controllerFile`` as a variable name because that's weird.

Also typically class names are capitalized..


Thanks for the quick release. I thought I'd have to merge the project and all those resource files manually to get that fix into my project.


Missing space before parenthesis.


I know, but somehow it was not working....

> On 9 juin 2015, at 02:39, Jay Oster <notifications@github.com> wrote:
>
> Really? me.Vector2d already has this: https://github.com/melonjs/melonJS/blob/2b2fc9a63add8596581e6123529fb7a509a501ba/src/math/vector2.js#L22
>
> —
> Reply to this email directly or view it on GitHub.
>


@SamTay, I was looking at this incorrectly earlier. You are correct. We should only have three parameters.


I noticed a potential issue with this test.  The `inv_hash.cmp` file has the vw version at the top:
```
Version 7.10.2
```
So this will require renewing the file every time we up the version. Seem like it would be better to add a:
`tail -n +2` to skip the first line in the command and ref-file.

Also: there are 2 very long lines listing all the possible UTF8 pair combos even when the input is minimal:
```
 4371 pairs:     !  "  #  $  )  *  +  ,  -  .  /  0  1  5  .... (very long line truncated)
...
4371 interactions: len: 2 interaction:    len: 2 interaction:  ! len: 2 interaction:  " len: 2 interaction:  # len: 2 interaction:  $ len: 2 (very long line truncated)
```
I wonder if we could output only the pairs that actually exist in the input instead of printing all possible pairs?


Not exactly. The second path array element is specifically to identify a controller path, as you can see from the ``setController`` method. The way it is currently set up, I'm guessing he'd end up with the word "controller" in a lot of his URLs.


It's good. go ahead.


Thanks, Nathan.


When I write a programm at work I always go for null-checks. I do not know why we do not have them.


I was just saying its not used by ContinuousJobsManager anymore - I was wondering if the function then didn't belong in the watcher class instead. However, it looks like the implementation needs to call ListJobs, so it can't be moved. Unless the watcher got a reference to the manager - then it could call what it needed.


Whitespace change, brace not needed.


Brace not needed.


Remove this blank line.


TeamCity Engage Employment :: CI [Build 24](http://teamcity.engagesoftware.com/viewLog.html?buildId=263&buildTypeId=EngageEmployment_Ci) is now running


TeamCity Engage Employment :: CI [Build 24](http://teamcity.engagesoftware.com/viewLog.html?buildId=263&buildTypeId=EngageEmployment_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:18


GetNextInterval?


This doesn't seem right... Whenever I load up my BL world, the lag does not improve upon loading new chunks... May I suggest adding about 20% more lag to it? Also, we should reduce the frames to 30 max, to give it a more cinematic effect.


@alandekok sorry, but still not working...


why does this macro need to be in the public header? can we put it in zookeeper.c instead? if really needs to be public, we should probably namespace it (i.e.: ZOOKEEPER_IS_CONTAINER) to avoid clashing someone else's namespace.


Why was this changed?


@lemire, great :) I've put this on my todo. I think picking some useful parts from SSEPlus would go quick and were also compatible with the license.

Thanks.


Yes, it was either passing the jobs manager or a delegate, I used the delegate approach.


![screen shot 2015-06-08 at 11 49 42 am](https://cloud.githubusercontent.com/assets/10639044/8042579/10a0a6de-0dd5-11e5-9543-5bee6ebf0d17.png)


On Jun 8, 2015, at 2:52 PM, Jorge Pereira <notifications@github.com> wrote:

> @alandekok sorry, but still not working...

  Is it the same error, or a different one?

  The perl docs SAY it should work.

  Which version of Perl are you using?

  Which OS?


Looks like you need the pod volumes, and the pod container.


Looks okay to me, but we missing something with the default setting on observable vector?
https://github.com/melonjs/melonJS/blob/def8f5bf68095f979c6d610581ed873dc93cfc58/src/math/observable_vector2.js#L65


:tada: :balloon: :birthday: :clap:


this is just hard for me to follow....

we iterate over the pod's containers, for each iteration pass, we do a deep copy of the pod, on the copied pods container, we set a context, and then return.  after the return, we then set the returned context on the original pod.

is this really the best way to structure the code?  i don't get the need for a copy at all versus just a method to

```
applySecurityContextForContainer(...)
```

it looks like we can have multiple providers, but even so we appear to return immediately if the first provider was a success, and never evaluate the others.


reack.


reack


@teg it looks like the purpose of using k instead of r is incase `dns_label_undo_idna` return 0, which it will do if libidn is disabled.


I am confused by this, what's the point of multiple providers if the first one basically wins?

It seems like context is never nil if err is nil, so the first provider of the set better be the right provider.


There should've been only two places, one on the main page and the other one where you fixed it.


Does the order that providers are evaluated matter?


See erlang/otp#769


@weltling

Sure. It would be best to avoid complicated dependencies however, since C does not have a great portable way to cope with that.


TeamCity Engage Employment :: CI [Build 25](http://teamcity.engagesoftware.com/viewLog.html?buildId=264&buildTypeId=EngageEmployment_Ci) is now running


That's what we'll start with. I'll probably have something tagged for testing in the next few days.


TeamCity Engage Employment :: CI [Build 25](http://teamcity.engagesoftware.com/viewLog.html?buildId=264&buildTypeId=EngageEmployment_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:13


@pretyman Can you please rebase it. It will be much easier to review for me.


From a readability perspective, it seems like having a method

```
getSecurityContextForContainer(pod, container, constraintsProvider)
```

is easier to follow than a method that works on a relative index and list of providers in a map?

Maybe have a parent func that calls that?  The for-loop stuff makes it kind of tricky.


Thank you both for your replies.


Windows 7


Is that what really happened?  You copied a pod.


You could just let it fall through to the IEnumerable part. String is enumerable, and wouldn't enumerate if the string is empty.


no se como empezar.


how many are these in practice?

is there a way you can index this instead of iterating a full store?

if there is just 1 of these in the store, no big deal, but if we expect 100, we should probably have a way to find a way to index this.


Bien.


I think this can be cleaned up.  The contract of CreateSecurityContext/ValidateSecurityContext was working directly with the SC on the container.  Create can always work with a copy and consumers can swap what is generated with what is actually on the container.  In our case we can revert that swap to get back to original values as we iterate through our SCCs


^ para reducir nuestro code, definí el dafault layout como "post" y el category como "ed/tech" (que es la categoría en donde más contenido hay que generar).


rax retry tests


Correction: 24 FPS are the cinematic optimum


Why not delete `-s branch=ref`?


@wkennington no idea what's the best way, but the current state is broken.


This breaks with a "list index out of range" error on non-windows systems because there is no \ in unix paths.

Looks like the better way of doing this is:
import getpass
getpass.getuser()


@simensen yes that is my goal, will check it out later and make a PR so we have proper 404 support :).

P.S. My 404.html:
```html
---
layout: default
permalink: no
title: Four o four
---

<section class="archives">
    <div class="container">
        <div class="col-lg-10 col-lg-offset-1">
            <img src="/images/404.jpg" />
        </div>
    </div>
</section>

```


I'm curious what the call stack looked like...


no, we use it to create the main menu


entrega


That will be difficult, so don't bother.

I'll have an Ubuntu O/S up later today and will test an updated version (using memcpy instead of an assignment).

That loses some of the benefit of the patch, but  still a bit more clear than V() and individual byte assignments.


I doubt this will crash on ARM. the pack #pragma seems to do what I want. See
https://gist.github.com/ckuethe/2837877eedd45bc375b8


Please add a `.` at end.


How about 3-6?

----- Original Message -----
> how many are these in practice?
>
> is there a way you can index this instead of iterating a full store?
>
> if there is just 1 of these in the store, no big deal, but if we expect 100,
> we should probably have a way to find a way to index this.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pweil-/origin/commit/40210f6677e0c9e6d44de1ea8bf31b8e0cd4e10e#commitcomment-11573870


Are you sure?  This was pulled verbatim from your original patch.  The idea was just to catch the case where it would under flow due to `arc_c_min` being to low.


On 06/08/2015 12:07 PM, ckuethe wrote:
> I doubt this will crash on ARM. the pack #pragma seems to do what I
> want. See
> https://gist.github.com/ckuethe/2837877eedd45bc375b8
>

The kernel must be trapping the unaligned access and converting
to a byte-wise copy.


can you use a namespace name as other than default?  I hate seeing default in a test because it leads to bad assumptions.


Ahh. Yes. The things we learn everyday!

I will look for your changes when I return to the PC. I'm stuck on a tarmac with only my mobile phone.

Kristen

-------
kristen kam

sent from Kristen's smarty phone..


Make this in a ```Namespace: other```


The cake is a lie


Plain ReactiveMongo apis => ReactiveMongo base API


extra allocation here, use `QueueUserWorkItem(Action<object>, object)`


Good catch, yes it definitely should.  I'll refresh my branch with this and any other feedback.


Looks great.


The operators should be ok, I took them from our script function GetRGBaValue() in System.ocg. I can add the parenthesis of course.


Why we need these two permissions?


Maybe this is a good use case for initializer pattern post 3.0 and kube 1.0 so we can avoid needing to compile in admission controllers in kube that need to invoke openshift resources.  will keep this in mind as we go forward.


Not sure why `eval`ing a path passed in by a user would cause issues. I'm a bit hesitant to add special handling for `~` and other characters and would prefer to let bash expand them itself.


Updated the gist, no crash when using the BE32 macro. If the kernel is willing to do the dirty work for us, let it.


3 is the maximum: the values range from 0 to 1.0, so 1^2 + 1^2 + 1^2 = 3


As for first - good catch, I'll try to fix this with tail.

As for 2nd - I would left it as is. Noone actually knows which interactions were used unless you collect this info in vw code. But as inv_hash is just a text version of regular model file and model file holds all possible interactions (you may face with them during testing even don't used some at training) I would left them untouched. Is this a problem to have lines that are too long? Or it's because of inv_hash.cmp filesize?


Point 1 is not yet done -- I could try to improve that aspect, yes.

Point 2 has been solved weeks ago -- Do you confirm?


1676-AirToAirTranslation (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Part of the contract of GenerateSecurityContext is that it will not touch anything already set. So what happens here is that P1 sets defaults for anything it has configured and then relies on ValidateSecurityContext to catch things that are not valid according to its rules

Example

1.  Two providers P1 and P2
1.  P1 has a mustRunAs user strat with uid of 2, P2 has a runAsAny strat
1.  Container C1 comes into this method with an SC that is not requesting anything
1.  Container C1's SC gets populated with defaults from P1 (uid 2) and validates against it, the SC is set for C!
1.  Container C2 comes into this method with an SC specifically requesting to run as UID 1000
1.  Container C1's SC gets populated with defaults from P1 (uid is not changed since it came in on the request)
1.  C1 fails validation against P1.Validate since it has a MustRunAs uid strategy with value 2
1.  C1's SC gets populated with defaults from P2, C1 validates against P2 since it has a run as any

Let me know if that doesn't make sense


Why new "menu_enable_dialog.xml"? We don't need it.


@lemire yep, i meant just to pick the functions needed, those two you've mentioned for the start. Including the whole SSEPlus were obviously an overkill. And even it wouldn't exist, it could be implemented. Then this emulation layer could be extended incrementally by need.


Ah of course, I was mistaken.


Love this @tnj!


bull


@knbk Where did the admin urls go here? -- isn't that the root urlconf?


In practice probably < 5.  In order to determine what matched we'd need to build an index based on the []users and []groups of each SCC so we could easily look them up.  Probably doable as a follow up just in case someone gets crazy.


you need to add arrays


In this case however, b0 should always have the top 24 bits clear, so it should not be a problem either way.  If it is failing here, the cause must be something else.


On 06/08/2015 12:12 PM, ckuethe wrote:
> Updated the gist, no crash when using the BE32 macro. If the kernel is
> willing to do the dirty work for us, let it.
>

Agreed. I do want to test this on one of our boards before merging though.


This code doesn't run over each provider based on how I read it..

```
for providerName, provider := range ... {
  ...
  context := ...
  if len(errs) != 0 {
    return context, providerName, nil
 }
}
```

It looks to me that we could have a setup with Provider P1 and P2.  P2 may error, but P1 may succeed, and by luck of draw, which ever P comes back in the for-loop first will win the day...  I must be missing something obvious, if so apologies


no


http://en.wikipedia.org/wiki/Builder_pattern

http://www.javapractices.com/topic/TopicAction.do?Id=29


Android studio reordered some stuff...


I guess if this is done as an initializer in future that can review the pod before its scheduled, at worse, it would fail the pod with an appropriate status... that seems fine if you had a proper event generated in response.  trying to capture future reqs.


Right!


Make all attributes final.

Also, use:

private final boolean am;
private final boolean active;
private final boolean vibrate;

instead of the Strings you use.


Sure


ok, not super concerned


Should'nt the interrupt() be part of the catch? (as suggested by http://stackoverflow.com/questions/3976344/handling-interruptedexception-in-java for instance)


Hello Brady,

No.

Lines 147-150 and 177 are only concerned with year and month.

Only the year and month elements of the value of $occurance calculated in the lines 148 and 177 (mktime) are ever inputs for further processing (and only in line 154). The day component never becomes an input for anything at all (except explode function). So it is irrelevant.

(Also the only values of $nd that are important to lines 148 and 177 are  29, 30 and 30 since these will cause an overflow to the next month)


Die sout was ik al kwijt :+1:


What if this was a boolean from the very beginning?


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


oh my , you're right fixed with a new commit


Argh, PHP 5.3 again...


was this missing a test?


Looks Ok !


ugh, wrong repo, fixing


Looks good..


Using private pub...cool...just read about it, wasn't aware of it...


No, you're right, it isn't meant to run over every provider.  It is meant to find the first one that validates with no errors so it only continues if it was unable to validate.  In my example, if P2 were in front of P1 then both pods would have validated against the RunAsAny strategy because that user has permissions to do whatever they want.


@bhklein heads up


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Thanks for noticing! Fixed


You should've had 0 -> 12.Your code would've been:

int hour = hours;
if (!am) {
  hour += 12;
}
time = hour * 60 + minutes;

// which still sucks. I hate the data types. Lookup JodaTime


Thanks. I'm happy you find the library useful.


Oh, and these methods are not in the right place here. If this is a data class, than that's the only thing it should do.


The call to `rangesByKey.Len()` needs to be inside the store.mu.RLock().


Are you kidding with this switch?


It was missing completely up until a few days ago, so yes.


I don't even want to review this class. It's that bad.


@smikes I've never worked with cscript before--could you explain why this is necessary?


Apart from the GUID function, this looks good. How does it look on your local copy of the server?


This addresses issue #6.


In other words, if I make no SC requests on my pod definition I will 100% validate against the first SCC provider.  If I make requests on my SC I may validate against provider[n] or no providers.


Another sign we need a (more complete) fuzzer for this kind of thing.


It's not! I think you have to use either FIND_PACKAGE(CUDA) or INCLUDE(FindCUDA). This is probably an incorrect merge.


rax retry tests


I feel like this could be cleaned up a little


I have added a corrective commit: https://github.com/trufanov-nok/vowpal_wabbit/commit/3aaf0771a048df78ca2cc6699c79807444a66967
Note that there are multiple entrances of inv_hash.cmp filename in single command line now (in test 88). Suppose in planned file\reference file matching algorithm (based on filename equality instead of file extension) you'll need to ignore filenames that were already encountered to filter out such duplicate names.


There is code that already does this. Whenever you find yourself doing something tedious but trivial, somebody already did it for you.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/b95526951f149120497d1895e77e45a30845ede8/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/b95526951f149120497d1895e77e45a30845ede8/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/b95526951f149120497d1895e77e45a30845ede8/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/b95526951f149120497d1895e77e45a30845ede8/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/b95526951f149120497d1895e77e45a30845ede8/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


This class does too many things. Try to split it up and make it worth reading.


Take it for a spin. Please use the 5.4 branch


Looks good!


boxed_number_simplifications (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@yole
Please take a look at the following commit
https://github.com/JetBrains/kotlin-web-site/commit/38cad4306a7f8d16b5299d45741a58cacd45631c


merge the same


:+1: If it compiles, ship it! ;-)


You find some really tedious solutions. Try to simplify things by using the right data structures.


sweet!


Django 1.8


Yes, most of the commits were just quick saves when I found something that worked. I tried squashing them where it made sense, without being self contained. The other option was cluttering the code with even more comments.


I saw a mix of `console.log` and `debug` in the tests, so I was trying to preserve the "debug level" of the messages.


Makes sense. IMHO this is an example of why it's good to be explicit about intent in the commit log. If the long term plan is to make check-noquattor=1 be the default, and check-noquattor be the legacy behaviour, then --force-quattor makes sense. If check-noquattor is an optional switch that most sites will not use then --no-check-quattor would be a better option.


Ok, I just copied stuff from elsewhere, because I had no idea how the colors are stored internally.


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/13118/testReport/) for this commit**


Just curious. How many data point do you want to take before calling it working?


no space after "@" would make it aligned with our code better


Was there a PR for this and https://github.com/saltstack-formulas/hadoop-formula/commit/854d9dc752c3a82eb6926430a7e6f90cc80305cd that I am missing? I don't see one.


what is the # &&& for? (here and below)


Why remove the flash11?


Is there a reason that decode_ciphertext returns the associated type Error rather than a ReceiveError::DecodeError itself?


Right


OK


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/15971/testReport/) for this commit**


Its unused?


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/13119/testReport/) for this commit**


We could delete this config.


merge away!


How did this ever happen?


So i just have to add one string after " 'amount'        =>  $tx->amount / 100," and it will work ok, and will not ask customer to pay for unlimited times?


@aembler What about renaming `conversation.banned_words` to `conversations.banned_words` too?


Nice, this is going to come handy in the future.


stage3d and native tile renderer will work similar, so no need in those checks


Ok, I overlooked the int. There is problem on retyping of byte to int - into next() and other methods should be added `& 0xFF`.


boxed_number_simplifications (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@xeux  in sonata_block congfig you can enable stats

```
sonata_block:
    default_contexts: [sonata_page_bundle]
    blocks:
        sonata.admin.block.stats:
            contexts:   [admin]
```


i'd do that in "debug", not "info" (unless you want to show these things to users by default)


Yep, this looks good.


Thanks for feedback. I changed to .seq files and the comment seems wrong or outdated now! s" foo" load still works fine, also with .seq files...


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/15972/testReport/) for this commit**


Is it a bit more efficient to pass an integral value instead of a pointer?


Do we have an example of how we can hook log4net/nlog into the new abstraction??


можно брать задания клиента прямо из self.clients_ids


Surname


I believe once the window gets on screen, I never tested it though.  Just that in the ``add_element`` method, it would return h=0 in the ``FigureManager.__init__``, so I had to give either a predefined size if the mpl toolbar/statusbar had a default value, or make an educated guess.  I did something like the following:

```python
if isinstance(element, Toolbar):
  h = 200
elif isinstance(element, Statusbar):
  h = 10
```

with the hope we could clear it up with MEP22


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


BTW, one better introduce the function freeIndex and call it after each run.


Nice commit
![squirrle-2](https://cloud.githubusercontent.com/assets/3319507/8043651/7cd3822a-0df5-11e5-8901-b719d9050267.png)


@mguterl no idea!


Couldn't we just reject hashes where cs.salt[3] != cs.vsp.domain_name_length?
These hashes should have been created by pcap2john (or vtp2john for older john versions).
If these two representations of the same value don't match, why pretend the hash is valid?


perfect


You have to remove division by hundred.
Red highlight shows before, and green highlight shows after state of the code


should we use temporary files here?


contributors*

I've never been good knowing about the need for the comma between "new and old". necessary?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.251](http://teamcity.monogame.net/viewLog.html?buildId=8148&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


From what I understand this only returns null if style.display == none, so it would not take effect when wanting to skip the contents of <script> tags or hidden elements of the page. Am I understanding it the wrong way?


Здесь лучше https://ru.wikipedia.org/wiki/%D0%A1%D0%BF%D0%B8%D1%81%D0%BE%D0%BA_%D0%BA%D0%BE%D0%B4%D0%BE%D0%B2_%D1%81%D0%BE%D1%81%D1%82%D0%BE%D1%8F%D0%BD%D0%B8%D1%8F_HTTP#403


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@azturner Gotcha, didn't know it did that.  We're scripting campus creation so I wanted to make sure I understood the structure.


Здесь лучше код 403: https://ru.wikipedia.org/wiki/%D0%A1%D0%BF%D0%B8%D1%81%D0%BE%D0%BA_%D0%BA%D0%BE%D0%B4%D0%BE%D0%B2_%D1%81%D0%BE%D1%81%D1%82%D0%BE%D1%8F%D0%BD%D0%B8%D1%8F_HTTP#403


just a side note: this might look a bit odd. However, replacing by iterating over characters *and* checking if it is in the text, is actually quite fast.


now that `assert_can_ping` supports retries, we don't need the retry code repeated in every test.


So far so good - one weird thing though,  I feel like I can see the armor stands for a split second on spawn- but I don't recall seeing this for explosions. Does that make any sense?


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


if this errors on any of the providers, you will never run the others, is that expected behavior?


this prevents the implementor from returning a misleading error. for example, they can't return an invalid message number error when it was a decrypt error. it also saves the implementor the trouble of wrapping the error.


bro this is useless


If you get an error, you should continue, and not return.  Probably log it as well.


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Actually all but one. But I changed that and made the change.


The update itself is done, I'm still working on porting the popup and menus to the new API:
![screen shot 2015-06-08 at 20 51 41](https://cloud.githubusercontent.com/assets/7221779/8043794/3d2cb47c-0e20-11e5-8094-cd461beb7af5.png)
If you prefer, we could merge this branch and I will open a new one for the UI work.


boxed_number_simplifications (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.72%25-yellow.svg)


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Strange. tota11y to the rescue!


Double space -> single space


Agree, as discussed in IRC, I think an error on provider.Generate is a candidate for removal from the interface definition.  Anything that causes an error in generate should be caught by the NewXXX method and should be fatal when initializing providers (accounted for in the main code line).


@kanitw What is category10-k? I'm reverting.


The library compiles with name `libgthree-1.so`, pkg-config feeds `-lgthree` to the linker causing an error.

It should be `-lgthree-1`


Are you *sure* it was me? :smile:


My env is:

```
[jpereira@jpereira-desktop ~]$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 14.04.2 LTS
Release:	14.04
Codename:	trusty
[jpereira@jpereira-desktop ~]$ dpkg -s perl | grep "Version:"
Version: 5.18.2-2ubuntu1
[jpereira@jpereira-desktop ~]$
```

don't have error, simple not show anything.

```
root@OI-WIFI-PROD-NV-RADIUS-1B:/etc/freeradius# cat dictionary.mobicare  | perl /opt/freeradius/freeradius-server-3.0.x.git/share/format.pl
root@OI-WIFI-PROD-NV-RADIUS-1B:/etc/freeradius#
```


Quacked ftw :D


On Mon, Jun 08, 2015 at 11:14:49AM -0700, Lennart Poettering wrote:
> Humm, no.
>
> The dbus spec specifically says an initial 0 byte should be sent that has the user creds attached via SCM_CREDS, and that that should be used for authentication. I am pretty sure we should not deviate unnecessarily from that:
>
> http://dbus.freedesktop.org/doc/dbus-specification.html#auth-nul-byte
It says "However, the nul byte must be sent even on other kinds of
socket, and even on operating systems that do not require a byte to be
sent in order to transmit credentials." It seems to leave the question
of what mechanism is to be used to transmit this metadata to the
implementatino.

One could argue that Linux unix stream sockets fall into this category:
both credential and security label can be retrieved from the connection
fd, without having any data.

> Moreover, I am pretty sure that if we use the SCM_CREDS info from that NUL byte we should also use the SCM_SECURITY data from it.

I think we currently (before this patchset) retrieve SCM_CREDS at connection
time in bus_get_peercred(). With the proposed change both credentials
and security label would be retrieved at connection time.

> It appears that the kernel is quite broken if it sends rubbish SCM_SECURITY for stream sockets. Either it should send valid SCM_SECURITY data, or it should not send it, but certainly no rubbish data. And if it does, that's something to fix in the kernel, and not in systemd.

True.

So the big question is: which identity of the peer do we want? The one
at connection time, or the one when datagrams are actually sent?


We could, but only i cs.salt[3] > sizeof(cs.vsp.domain_name)

The data is 'not' split into independent fields.  It is a packed 'structure'.  Why the data was put back 'into' a structure is something I can not really fathom.  It should have just been an array of bytes, and the decode simply did the hex decode into.

But it will be easy to add this check to valid.


From top to bottom:
* loginName renamed to userName (could be confusing because in all other SASL mechanisms and RFC is username/userName)
* authorization check moved after checking equality of gs-header in client messages
* parsing of `y` and `n` is identical in final message, so I merged it into one part of switch
* added  checking of equality of parts gs-header (removed checking as one string from base64, because I I overlooked before, channel binding data are only in final client messages)
* More meaningful name of `s`/`proofOffset` variable


un "layout" serait utile


Is OK.


hey- can you fix this merge conflict?


The fix for #1465 is incomplete:

```
pyLoad 0.4.9 Debug Report of ShareonlineBiz 0.52

TRACEBACK:
 Traceback (most recent call last):
  File "/usr/share/pyload/module/PluginThread.py", line 187, in run
    pyfile.plugin.preprocessing(self)
  File "/home/pyload/.pyload/userplugins/internal/Plugin.py", line 237, in preprocessing
    return self.process(self.pyfile)
  File "/home/pyload/.pyload/userplugins/internal/SimpleHoster.py", line 539, in process
    self.checkFile()
  File "/home/pyload/.pyload/userplugins/hoster/ShareonlineBiz.py", line 122, in checkFile
    return super(ShareonlineBiz, self).checkFile()
  File "/home/pyload/.pyload/userplugins/internal/SimpleHoster.py", line 599, in checkFile
    errmsg = self.checkDownload({r: re.compile(p)}).strip().capitalize()
AttributeError: 'NoneType' object has no attribute 'strip'
```


lol did Josh Owens hack us?


maybe "bootcamps and education".


This is a real header?


ToStringi? I'm guessing this would cause a compilation error.


Das sollte auch bei ner pipe kein Problem sein, denn die verbindet immer stdout und stdin miteinander, stderr wird weiterhin normal auf der Konsole ausgegeben.


Deployment for Testing

**run the apache server and put project files in htdocs and load index.html


Missing space before parenthesis.


@sunaot わざわざコメントしていただいてありがとうございます :bow:
良くないところなどもあればぜひ教えて下さい :bow:


images without background


this if block could juset be `WasClicking = true;` the if just lags things.


Thanks, fixed!


Why did you choose `long` as variable to store the light color? `long` is always 32 bit on Windows, but on Linux it is 64 bit on 64-bit systems. I'd suggest `uint32_t` instead, which is always 32 bit everywhere.

It shouldn't really matter though, except for 4 wasted bytes on Linux :). The many occurrences of `long` in C4Script.cpp are simply an artifact of the early days of the 64 bit port.


I was thinking that... But I was thinking it'd be worse to continue to set the variable.

Of course, this isn't Java so idk


const (
    connectTimeout = 3 * time.Second
    readTimeout = 3 * time.Second
    writeTimeout = 3 * time.Second
)

maybe more clear?


So `lightTextureSize` and `offsetColor` are not used here? They are probably optimized out, but could just as well be removed entirely...


committed


Besides hex digits and colons, some IPv6 addresses whose last 32 bits encode an IPv4 address may contain dots. See for example RFC 4291 sec 2.2 part 3. I don't think it's possible to come up with an exhaustive list of IPv6 prefixes where this notation is valid and where it isn't.

The theoretical maximum length of an IPv6 address given in this format would be 45 characters rather than 39 (six sets of four hex digits, six colons, four sets of three decimal digits, and three dots) if that helps your calculations. I don't know that we'd ever see such a thing in the wild.


这个也需要移除


Allow the user to leave the table prefix blank on install. Also validate the prefix with a regex if something is entered.


Why does default indicate the first pass? What I'd typically do in such a case when switching on an enum is to add a statement like

```
default:
  assert(false);
  break;
```

at the end, to make sure that for invalid enum values an assertion is raised.


do not use triple quote here `'`

instead use the "backtick" key, which is on the `~` key


Why don't you also call `glBlendEquation` here? Do you rely on it still being `GL_FUNC_REVERSE_SUBTRACT` from the second pass?


Also, for comparison and database storage, it might make sense to canonicalize an IPv6 address into the full exactly-39-character representation, but accept as user input (or CDN proxy header input) anything that can be converted into that format.


You should specify what exception you are catching


Also Camera orientation code from codepath not working, need to ask Nathan about it.


I'd keep the `case` statements in order: C4DP_First, C4DP_Second, C4DP_Color.


Yeah, display hook only half works like I thought/hoped it would :disappointed:

I threw print statements into the callback functions to see when they were called:

```python
22:14 $ python -i /tmp/test.py
True
>>> for j in range(6):
...     plt.plot(range(j+3)[::-1])
...
display hook
[<matplotlib.lines.Line2D object at 0x7f5e5b9db0b8>]
display hook
[<matplotlib.lines.Line2D object at 0x7f5e5b9d4940>]
display hook
[<matplotlib.lines.Line2D object at 0x7f5e5b989e80>]
display hook
[<matplotlib.lines.Line2D object at 0x7f5e5b98f668>]
display hook
[<matplotlib.lines.Line2D object at 0x7f5e5b98fe80>]
display hook
[<matplotlib.lines.Line2D object at 0x7f5e5b9946d8>]
>>>
```

```python
22:14 $ ipython -i /tmp/test.py
Python 3.4.3 |Continuum Analytics, Inc.| (default, Mar  6 2015, 12:03:53)
Type "copyright", "credits" or "license" for more information.

IPython 3.1.0 -- An enhanced Interactive Python.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://binstar.org
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.
True

In [1]: for j in range(5):
   ...:     plt.plot(range(j + 3)[::-1])
   ...:
display hook

```


That's a bit a generic name for a very specialized enumeration. Why not just move it inside the C4FoWDrawLightTextureStrategy class, so it does not clutter the global namespace?


Baldur's Gate: Dark Alliance FMV's seem to have garbage colors on screen, no idea what it is. If I switch to software then hardware mode, garbage color disappears entirely, but if I start in hardware, it's there.


reverted


If you add another value to the enumeration, you could return that here:

```
enum C4DrawPass
{
  C4DP_First = 0,
  C4DP_Second  =1,
  C4DP_Color = 2,
  C4DP_Last // gets automatically assigned the value 3
};
```

This way, in case the number of passes is modified next time, one only has to modify the enumeration, and not the `GetRequestedPasses()` function as well.


So I understand this commit is some intermediate progress, but playing the game with just this would not work properly?

I like it when commits are self-contained, and don't leave the repository in some intermediate or inconsistent state. This is especially useful when one tries to `git bisect` later.


@michelamaral Corrigido, porém agora a pagina não atualiza, e o nome do usuario continua vizivel até atualização da pagina.


In other words: It might make sense to squash it even more with some of the following commits.


well, well, well, you were right.


这是一个测试评论。


If you want to use your code, go right a head.  I'm just saying that since someone else has written the code, and is maintaining it, it would be less work for us to use their's vs rolling our own.


Are you sure about the operator precedence rules in `color >> X & 255`? I'd suggest `(color >> X) & 255`, even if just for clarity. Also, the call to `Min()` is superfluous; ((X & 255) / 255.0f) should always be between 0.0f and 1.0f, no?


so merge this to you master branch! IdiotZx..!


Just checking on commits


hàm sao lại viết hoa ???


rax retry tests


removed?


I submitted #5348 seconds after Seb made that commit! This is the same fix, will close the PR.


Codigo amil


Just for my understanding: The `normalize()` here is because when the texture is drawn, the interpolation between vertices performed by OpenGL might cause the value not to be normalized anymore?

If that's the only reason, it might make more sense to normalize the value in the shader that's used when drawing to the texture, not when reading from the texture and drawing objects. Did you consider that?

I'm also wondering whether there's a chance to use the normalization of this value as an additional piece of information, such as the brightness of the color (such as dark red vs. bright red). I know you have experimented quite a bit with this, but didn't fully follow the discussion... can you summarize why this is not used?


What if `r = g = b = 0`?


Actually which problem were these changes trying to solve?


Just because this may be true does not make it good practice... our intention is to compare by the actual value of the block type and not by reference. Using `==` doesn't really add much and imo makes things more confusing


Where does that factor sqrt(3) come from? Is it just experimental?


Should be davinci_coders


They are steps toward two goals: one is making the object-oriented style fully interactive, without having to call `draw()` all the time, and the second is simplification of the code base.


Thanks a 1,048,576.


なるほど、こうspecをかけばよかったのですね。勉強になります！


- - you have to create a full request for me at first..


funny indentation...

isn't the multiplication with `norm` superfluous because the value is re-normalized anyway in the shader? Hm, except that each color component gets clamped to 1.0, so the result would be different, but it's hard to tell whether that's a bug or intended... so some comments explaining the overall strategy here would be helpful, I think.


@jrfonseca, are you sure you want to declare the function with APIENTRY? I wonder if that's why it's not working for me:

```
Program received signal SIGILL, Illegal instruction.
0x68cc1635 in ?? () from C:\ProgramFiles.x86\Pidgin++\libssp-0.dll
```


See https://github.com/matplotlib/matplotlib/pull/4091  for motivation.

I have found a way to solve the original problem, in all of the backends there is a test when figure managers are created to determine if we are in interactive mode or not to determine if `show` should be called or not.  We just need to add a `canvas.draw_idle()` into all of those tests.

As for the second (raw python not calling display hook when evaluating statements) I have pinged the IPython gitter to see if there is an easy way to do that.


iColor is an `uint32_t`, so it cannot be -1. This will probably be converted to `0xffffffff`, which corresponds to white light. Maybe directly initialize it with that instead, it should at least avoid a compiler warning.


@klgilbert We'll need to check that the benchmarks work across the different Rails versions :smile_cat:


LGTM


I would add glScissor around this as well, otherwise you clear half of the texture twice, which is unnecessary work.


rax retry tests


Format improvement
![example xau_usd](https://cloud.githubusercontent.com/assets/6808438/8027489/f6a7f8c6-0d5d-11e5-8c81-0c6f3abe3c21.png)


Hm, I see some of the things I commented in the previous commit are fixed here already :)


@efiring  Sorry I am a bit lost. Do you have a simple example of an object-oriented script where you have to call draw() ? This sounds a lot like what PyOS_InputHook is doing, which is tricky, so I think we should be very careful here.


What if value=0, or makes any of r, g, b greater than 1.0?


hey, @junfan 樊老板
验证码好像不行


At the repl run:

```python

fig, ax = plt.subplots()
ln, = plt.plot(range(5))

ln.set_lw(5)
ln.set_color('purple')
```

and the plot should auto-redraw to reflect the changes to the underlying state of the objects. (the second one  won't work in base python, but behaves correctly in IPython).


Do we still need security checks à la `if(!Obj) return 0;`? I see many functions don't have them (anymore?)...


I went through the commits "bottom-up"... some things I commented on you fixed in later commits, or moved elsewhere, but I think most comments are still valid in the latest version.


This breaks builds for anyone not using
```
maven {
    name = 'sonatype-nexus'
    url = 'https://oss.sonatype.org/content/groups/public/'
}
```


Autocorrect changed "linting" to "listing".


Thanks! I've updated it based on your feedback and [submitted the PR](https://github.com/rust-lang/rfcs/pull/1154).


처음에 그냥 for loop로 짰는데.. 그렇게 하니 Intellij가 이거 Java8 Stream으로 바꿀 수 있다고 알려주더라구요 ㅋㅋ


This is temporary, I'm waiting for @gratimax to finish an overhaul of the noise library so I can deploy it as a new version to central. Right now we need snaphots for Skylands to work. Gratimax is busy right now with a lot of other stuff so this might take a little while, but I'll be discussing with him tomorrow to see if I can take over the refactor so it goes faster, as I have free time to work on it.

> On Jun 7, 2015, at 10:48 PM, Wyatt Childers <notifications@github.com> wrote:
>
> This breaks builds for anyone not using
>
> maven {
>     name = 'sonatype-nexus'
>     url = 'https://oss.sonatype.org/content/groups/public/'
> }
> —
> Reply to this email directly or view it on GitHub.
>


:+1: This is an amazing change, massive performance improvement for high numbers of polygons. Well done!


see https://github.com/matplotlib/matplotlib/pull/4503


Sorry to meddle in your waters, but shouldn't this call be `LibM.remainder_f32` instead?

:)


Fixed.


I would use SlimLauncher or Nova, Trebuchet is not nice unless you tweak it. Just my opinion though.


Hey @epgarcia @pavel-savinov

are we including this only for the namespace? If this is the cose, how would you feel about injecting the namespace in all the rules by default instead?


Do this:

$ cd share
$ ./format.pl dictionary.mobicare

On Jun 8, 2015, at 3:57 PM, Jorge Pereira <notifications@github.com> wrote:

> My env is:
>
> [jpereira@jpereira-desktop ~]$ lsb_release -a
> No LSB modules are available.
> Distributor ID: Ubuntu
> Description:    Ubuntu 14.04.2 LTS
> Release:    14.04
> Codename:   trusty
> [jpereira@jpereira-desktop ~]$ dpkg -s perl | grep "Version:"
> Version: 5.18.2-2ubuntu1
> [jpereira@jpereira-desktop ~]$
>
> don't have error, simple not show anything.
>
> root@OI-WIFI-PROD-NV-RADIUS-1B:/etc/freeradius# cat dictionary.mobicare  | perl /opt/freeradius/freeradius-server-3.0.x.git/share/format.pl
> root@OI-WIFI-PROD-NV-RADIUS-1B:/etc/freeradius#
>
> �
> Reply to this email directly or view it on GitHub.
>


Is OK.


@westcripp thanks for pointing me, how can I forget the arrays...


Сделайте в сервере возврат валидного JSON. Тогда в клиенте можно из строки response.body получить исходный dict или list с помощью json.loads(response.body): https://docs.python.org/3/library/json.html


Would you mind rebasing instead?


Hi Nigel. Thanks for the patch. Did you create, or is there, a Moodle tracker item for this issue?


Ignore the second change. Mistake.


boxed_number_simplifications (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.72%25-yellow.svg)


Hey @epgarcia @pavel-savinov

it would be very nice if we can have the url as the url for the icon, but then we use the onclick method to call the javascript logic instead of just following the link. This way, event if javascript fails it would work, or even doing ctrl+click for a new tab would work. We follow this pattern in other places in the portal.


This unit test didn't fail originally under #24377, but failed during one of my trial fixes, so I left it in.


I think you need some extra padding on this (i.e. the text is butted right against the lock now).


works fine, but don't complain the example in the docs.

#  Usage: cat dictionary | ./format.pl > new

anyway, We need to fix the "Usage"

thanks.


Is there a strong advantage to having an associated type for errors?

Rust conventions for error handling are still settling, but it seems like concrete Error types are favoured. (either std::Error or sum types), is there a benefit of making users implement there own?


fixed


Similarly the reader button seems really close too (load a facebook page with a long url).


@mkreisl

yep, squish I consider as broken in some way too. I fought with it each compilation.

but as I ignore v15 for now, do/adapt/edit/reorder those specialties as they will work with YOU and build system.


This test failed under #24377.


Also moved some stuff around to match the other file... might put them in alphabetical order by id later


Hi Arda, awesome to see you :)
If it is that easy I don´t see why not,  it is definatly much more optimised to use as little resources as possible.


Please create new release notes for this.


Tecla ao lado, deverá ser "por outras aplicações" :)


OK!


This test and the other "ignores_default_pks_on_submit" variants that I added never failed; I'm not clear on what code they're hitting (other than obviously not what I was working on), so I don't know whether there's value to having these cases to pair up with the "nulls_default_pks" counterparts or not.


Ah, sure that makes more sense. Thanks.


Leave error on; Helps me determine what's causing the server not to start.


Hi,
i'm curious as to why you're using closure compiler - it looks weird to me to depend on java for minification.
Is it so better that it justifies the inconvenient ?


This worked under #24377, but is the opposite arrangement to the test that failed (`test_inlineformset_factory_nulls_default_pks_uuid_parent_auto_child`)


Ma premiere contribution


I am not sure also in fact, I started to switch back to uglify on some projects. Feel free to PR if you want to switch back to uglify here :)


Comment at line 2 should probably be updated.


Oh, yeah, this one is a classic!  It is specified by the C++98/03/11/14 standards to do absolutely nothing! :-)  Makes it ideal to include it for version checking.


It occurs to me that I should fix the comments on these to reference the new bug number that I didn't have yet when I wrote the code.  I'll do that shortly.


@j2sol Should we push this upstream to ansible/ansible?


What happens if there is no ResourceType "Classroom"? Since it used to do a get_or_create. (Why are there multiple of them?)


I will update now and push up


Or, I guess if there is no such resource type then the result will be an empty set anyway.


This line results in the following error when initializing the database.

ERROR::MAIN :: DB error: default value of column [old_status] is not constant


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.251](http://teamcity.monogame.net/viewLog.html?buildId=8148&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:21:11


Don't worry, I'm working on. Soon enough we'll only be to able 5 whole frames per second!


Ah yes, commented wrong line - I meant it to be the replacement for the first part, not much important though.


"Registro" é grafia do Brasil, deveria ser "registo" :)


This needs more explanations. In my mind, saved passwords are kept in different password managers. If the user attempts to log to mysite.com with hat A, but the password of mysite.com is in hat B, this means that either:
- the user has different accounts on hat A and hat B; or
- the user is on the wrong hat and needs to be transferred to hat B; or
- the user really wants to do that, so Firefox needs to ask her whether she wants to import the login to mysite.com from hat B to hat A.

Apparently, we disagree here, so what's your take on that?


should _not_ return generic error message?


Idea: The ignore prefix could be read from .gitignore, if there's such a file in the base directory.


Hmm, you do have a point I guess, It indeed leaves open whether we should use the NUL byte.

I do wonder though whether it could be considered problematic if we break compat with libdbus in this regard. Hence, we really should check what libdbus and gdbus do in this case... if they always use SO_PEERCRED then we should too. If they always use SCM_CREDS on the NUL byte, then we should too.


This looks good, I had one suggestion that could be considered for a possible v2.


I don't like this at all. :(


ack


@mk01
Already did it :-)


did you ignore my comment about the result_dict?


Grafia do Brasil


s/ignore/oversee/


thanks to fix this game cold winter
but fix too legend of kay
a graphic bug on trees
resolution : fixes by (VU1)superVU recompiler


This is nice, although the obvious problem we have here is having to update this with changes, big or small in the options/options help. It's not something that we will tackle in this PR, so this looks good.


Sure.


On a system on which ```arc_c_min``` is 32MiB, if ```arc_c``` ever falls to exactly that value, it will never grow again due to the ">" operator.  I got this wrong in my original patch.  I suppose the alternative would be to floor ```arc_c_min``` a bit higher but I think the ">=" logic makes more sense..


Grafia do brasil: Registros -> Registos


Replaces https://github.com/projectatomic/ContainerApplicationGenericLabels/pull/22


Yikes! I forgot about this situation. Neither the original solution, nor your solution is right I think (if I understand what you have done).

*) The array is passed by reference, so reference equivalence must be checked (normalised according to alias in the normal way). If the reference equivalence is not checked then the unaffected part can detect the change in aliasing structure using the == operator.

*) Arrays from one partition can be read by the other partition (once passed across the boundary), so (as you observed and was not implemented) the contents of the array (and in fact its length) need to be checked for equivalence. However - arrays are mutable, so it is not sufficient to check them when they are passed across the partition boundary. Arrays must be checked whenever they are accessed. This means that arrays need to be labeled with a partition, and partition crossing array reads and writes (including reading the length) should be treated as method calls (i.e. should be incuded in the trace and checked for equivalence).


You'll have to update `trusty` -> `utopic` here. I've updated my repos recently.


#include<IOSTREAM
>


Dictionnaire : Les parenthèses délimitent les tuples, les crochets délimitent les listes et les accolades {} délimitent les dictionnaires.


Good.


What is the benefit of splitting this into a separate class? Normally the benefit is that we can't modify the original class or this class has extra dependencies and will be in a separate JAR from the original class. But that doesn't seem like the case here.


MIT currently has multiple ResourceTypes with name "Classroom" and IIRC it was causing me some problems, so I changed it to this. If there are no Classroom types, this code will return that it isn't scheduled anywhere, which is intended behavior.


我翻了好久才在最低下找到真正添加的代码，其它都是ISE自动生成的吧，喂！原来github这么笨啊，我的ccs工程也有这个问题，其实很多代码都不是自己写的（跑），显示更新了几千行，呵呵哒


Add the 3 buttons you deleted back into the Automation Menu


I think you can drop `two` here, as we might operate with more than two different types of tests eventually (vt-compat, for example).


Do these last four attributes hold value? I don't see them in the [jar specification](http://docs.oracle.com/javase/7/docs/technotes/guides/jar/jar.html#Main_Attributes) and I've not seen them used before.


how about we use `check-ccm-disabled` instead of `check-noquattor`?


lol ;)


Без теста? ;-(


lgtm, thanks for fixing my typo.


Is this an array or a typo?


Sure, the reorganization makes sense. At first I thought configuration had to feature in the very beginning of the docs, but I find your reasoning better. Also, avoiding possible confusions with the concept of SimpleTest and a basic example is a nice touch.


rax retry tests


No need to have multiple test files for your test cases, You can declare multiple classes in one file, thus simplifying the reading of the testcase.


Cool.  We need to fix travis.ci to pick up on these things.


Hmm... this was purposefully internal-only and only for optimizations. Again, I question the value of MetadataKeys, as I don't see what problem it is solving.


Well you better read the inline comment I have there and do not revert.


ok, dann kann man das zur not auch wieder wegnehmen


what do you mean?


This probably misses a break statement, once this variable is set to false, no need to continue with the loop.


each algorithm is run in a separate process so that memory allocation is not a problem


Ahh I see.  So what exactly are you proposing.


Same remark about only one file with all your test cases : it will simplify maintenance.


Why this change? It looks like `description` is guaranteed non-null due to the augmentDescription.


Hmm. Vega didn't find it. But it's down there... hmmm.. I'll check.


Change the operator on line 3359 to ```>=```.


(This was originally just a proof-of-concept, and implementation of the infrastructure for doing it, so the section titles where just place holders).


@afx114 pointed in IRC on problems with this code. Though this shouldn't influence because expected values: true/false, but shell variable must be escaped and not a x-trick.


you don't need the ```it```, you can just say
```
const VectorType currentPoint(*(unmirroredPoints.begin());
```
and remove the superfluous declaration in line 350


The total value is being shifted by 8, but that is with the deltasum included. I think the amount of shifts should be decreased to get the same D gain range.
Will play with it anyway


use ```const VectorType```


Sure, as discussed in the meeting this looks good.


@cpennington I have added calls to `modulestore().bulk_operations()` and updated this tests. Do the query counts look good now?


Missing an `s` here and the filename.


I think cancel should probably put back the original url text, regardless of what you've done. Maybe that's what this does?


This is meant to implement the second request from http://aegidian.org/bb/viewtopic.php?f=6&t=17477.  It's not finished, but feel free to have a play - the </> keys on the chart or system info screen will switch the info system along the ANA route.


I have to fork and stuff, too much hassle


I'm deleting my batch file, leaving this here as a reminder of what it contained.
```bat
mkdir build || del /s /f /q .\build\* && for /f %%f in ('dir /ad /b .\build\ ') do rd /s /q .\build\%%f
cd build && cmake -G "Visual Studio 12 2013" -T LLVM-vs2013 -DCMAKE_CXX_FLAGS="-D_HAS_EXCEPTIONS=0 -Qunused-arguments -Xclang -std=c++1z -Xclang -Wno-overloaded-virtual" -DCMAKE_C_FLAGS="-D_HAS_EXCEPTIONS=0 -Xclang -std=c11" ../extensions/Template/ || pause

```


An event is optional and can be more than one delgate, in this case I want it required and a single.


Sure looks like an odd case. I think this check does at least not cause any harm and can be merged as is, but I think @rcombs should take a look at the surrounding code and look if something doesn't make sense. (AFAIK he wrote or refactored this part.)


Hmm.  It shouldn't call vega's palette.
Jeff suggested that we should just have our own set of palettes here.

Anyway, gotta finish cleaning. Ttyl.


That's weird... Why there no default operator priorities?


Here same story.....divided by 32 should be something like 10.


I'd add "If you want multiple tests, (more on that later), we recommend you to follow the PEP8 convention for function/method names, lowercase_with_underscore".


should use `$location.host()` rather than doing this magic in here


aunav 加入子選單


This breaks many of plugins that are using this command API :(


You forgot to define it in serial_cli.h


@TheMolkaPL The old command API is still in WE after this commit.


I think the url isn't selected anymore after tapping it with this. (our tests aren't good enough here...)


Ouais c'est franchement mieux


This commit will help those of you trying to install SCOT on RHEL6.  It probably still has bugs, but using I was able to get a working install.  Please submit pull requests if you find problems in this.


Ref: #2


The height appears to be set twice. is that the cause of the chat box and the input box overlapping in             http://agar-clone.mybluemix.net/


So how should it be like?


Is there a continuous version? Probably this adjective is excessive.


Right, makes sense. I guess this will anyway become less of a problem once systemd issue #104 is resolved.


Looks good to me!


We'll have to remove these. Longer term import XCGLogger and private let log = XCGLogger.sharedInstance() at the top here will let you keep logging you want around longer term.


Okay, now I know what happened. you renamed it and removed the dash. However, the dash appeared again so this broke. I fixed it. Thanks.


This is just crazy


@fishcakez how do you feel about this approach? We're getting `ExUnit.LoggerFormatter` by default and have no `Logger` tests failures at the same time.


Right.

What do you think about mentioning that "out of the box" we have these two types, but extensions can add more types?

I kind of have the feeling that, on a Reference Section like this, it makes sense to document what Avocado (core) does.


make it more clearly for reading.

![screenshot_1](https://cloud.githubusercontent.com/assets/10940794/8044696/9e9ee05a-0e2e-11e5-8ef3-b38281d8dbdc.png)


If the long term plan is to make this the default setting and then /etc/noquattor will actually disable both ccm and ncm-ncd then I think check-noquattor is just right.


@mikesaurio la variable `correo` es mejor usarla en inglés para mantener homogeneidad del lenguaje


I think it should be public as well. If someone changes `p` for some reason, `q` should be updated as well.


Longer term (i.e. I'm not sure how hard it will be) it would be nice to just have constraints for these instead of these insets (then we could animate the changes here).


remove the useless code


I think there is no need to mention `q` as it is not in the argument list.


`apply to all programs called by the test` seems a tad more consistent.


... or even
```
const VectorType& currentPoint(*(unmirroredPoints.begin());
```
to avoid even more copying


I'd love to spend some time shopping for sphinx extensions (or more time writing some) to cover these situations. For now, let's live with that.


Do you think we should use `i32` or `i64`?


OK!


ctx.Destroy() can fail, you should check for error, because if it fails, user will get unrelated error. Same with the second ctx.Destroy()


@SzymonPobiega bye bye dynamic casting trickery


GetNextOccurrence?


use `multiplyScale` here


This looks related to https://github.com/ShaneWilton/sublime-smali/issues/4? What does it do?

Does it allow Sublime to correctly recognize comments?


Wow


This looks related to https://github.com/ShaneWilton/sublime-smali/issues/4. What does this change do? Does it allow Sublime to properly understand comments?


Ooops! I'm the GIT now.


*chuckle*. Approved!


Pretty sure that can't actually happen and the analyzer is complaining because it doesn't realize `is_new_bm_run` returns 1 on `!last`. Might want to report it as a false positive in the analyzer, unless it's reported a specific path where this actually could happen that I'm missing. Either way, the check probably doesn't hurt.


Also, probably an unsigned type would suite better. What do you think?


nit, space before the {


@mikesaurio un par de comentarios sobre este chunk de código:

1. Este tipo de queries `User.find(@complaint.user).business_name` no debería estar en el view layer. Solicitas múltiples veces el User.find y provoca más consultas a la base de datos.
Si desde la clase del mailer le mandas a esta vista @user podrás ahorrarte varios queries.

2. Las variables están mezcladas en inglés y español: i.e. `city`, `razon`, `domicilio`, etc.


This commit breaks the latest release i

```
 [2015-06-08T20:37:14+00:00] ERROR: apt_repository[percona] (percona::package_repo line 19) had an error: Mixlib::ShellOut::ShellCommandFailed: execute[install-key 0x1C4CBDCDCD2EFD2A] (/tmp/kitchen/cookbooks/apt/providers/repository.rb line 28) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '2'
       ---- Begin output of apt-key adv --keyserver hkp://keys.gnupg.net:80 --recv 0x1C4CBDCDCD2EFD2A ----
       STDOUT: Executing: gpg --ignore-time-conflict --no-options --no-default-keyring --homedir /tmp/tmp.7q9G4wTQTP --no-auto-check-trustdb --trust-model always --keyring /etc/apt/trusted.gpg --primary-keyring /etc/apt/trusted.gpg --keyserver hkp://keys.gnupg.net:80 --recv 0x1C4CBDCDCD2EFD2A
       gpgkeys: key 1C4CBDCDCD2EFD2A can't be retrieved
       STDERR: gpg: requesting key CD2EFD2A from hkp server keys.gnupg.net

       gpg: Total number processed: 0
       ---- End output of apt-key adv --keyserver hkp://keys.gnupg.net:80 --recv 0x1C4CBDCDCD2EFD2A ----
       Ran apt-key adv --keyserver hkp://keys.gnupg.net:80 --recv 0x1C4CBDCDCD2EFD2A returned 2
       [2015-06-08T20:37:14+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
```

Also changing the namespacing is a backwards compatible break. Not sure if that was intended / desired? Easy enough for me to fix that on my end but I was overlaying the ubuntu keyserver via `apt_keyserver` and a different `apt_uri`


It's hard to say exactly what the problem is, but with our version of this (which is almost identical to this), we end up with dead symlinks depending on the order of stable vs unstable trees on subsequent builds.


`Avocado at its simplest configuration can run two different types of tests (footnote - Avocado plugins can introduce additional test types)`.

Something along these lines seem reasonable.


@johncrossman thanks for removing the truthy.

![image](https://cloud.githubusercontent.com/assets/746759/8044827/2ab2dca4-0de4-11e5-9275-f1fd74dd9cb7.png)


Should it not be profile value?
{ "dterm_cutoff",               VAR_UINT8  | PROFILE_VALUE, &masterConfig.profile[0].fcut, 10, 50 },


Hi @navarr,
I agree that this solution doesn't have to be perfect. Especially if that adds a lot of overhead. But it's important to me that there aren't any false positives. I don't think it's safe to ignore these in between whitespaces changes since they might actually make a difference.

I feel the solution of `trim()` line by line (and maybe deleting empty lines) is 'good enough' even if it might still show any in-between whitespace changes (better show them as ignore them).


ты прав, надо бы покрыть


Hi. Can you add a new Moodle tracker issue for the questionnaire component for this? I think for this to remove the deprecation warning, there needs to be a new 'standard_intro_elements" method added, doesn't there? or is that part of the moodleform_mod class? If so, do we even need the "method_exists" check? We should put this into MOODLE_29_STABLE only.


Yeah let's do that.
On Jun 8, 2015 9:53 PM, Marco <notifications@github.com> wrote:The update itself is done, I'm still working on porting the popup and menus to the new API:

If you prefer, we could merge this branch and I will open a new one for the UI work.

—Reply to this email directly or view it on GitHub.


Good point)


Hey @Domi-php , nice to see ya :)

Firstly, keep up the great work @hien- :+1: :smile:

Actually `gulp` is both easy and beneficial and not.

**Easy and beneficial because:**
* This encourages, and guarantees that all 3rd party assets are in newest versions while building.
* You can split app resources into separate asset folders like I did: https://github.com/Ardakilic/WhatTheTag/tree/master/resources/assets . There isn't a big `jscript.js` file for the whole application in sources, they can even be separated for each logic. But in the end they are all merged, compressed, minified, if they are less or sass files they are also converted to css. E.g: every assets (including fonts) are generated with the gulp task in this folder: https://github.com/Ardakilic/WhatTheTag/tree/master/public
* After creating a gulp task, with a simple command (`gulp`) you fetch all assets from downloaded npm repositories, copy them, minify them or merge them, do whatever you like.
* This will keep front-end developers and theme designers happier and make the script help keep the assets up to date easier (`npm update`, `gulp clean` and `gulp`. You're done, you've updated all your assets and put them into specific folders) (imagine theme specific assets are merged and minified along with core assets :dancer: )
* There is also a watcher built-in. Let's say you are editing the source javaScript for a new feature, or created a new one, it listens, understands it and auto updates / merges etc. the specific content.

**Not easy beacause:**
* A shared web hosting solution cannot install or run npm, gulp and their dependencies. The release should provide precompiled assets.
* If this `$fusion_jquery_tags` variable has a PHP-generated variables, the logic of these variables should be changed so that these values are fetched from attributes.

E.g: let's say there is a code like this (to be honest I don't know currently if there are any lol):

```php
foreach(...) {
?>
<a id="loadContent<?php echo $data->id; ?>">Click Me</a>
<script>
$('a#loadContent').click(function) {
$.load('/ajax_call?id=<?php echo $data->id ?>');
}
</script>
<?php
}
```

The logic can be changed using attributes:

```php
foreach(...) {
?>
<a data-link="/ajax_call?id=<?php echo $data->id ?>" rel="loadContent">Click Me</a>
<?php
}
?>

<!-- this is now below the loop, now this is a PHP-undependent asset -->
<script>
$('rel[loadContent]').click(function) {
$.load($(this).attr('data-link'));
}
</script>
```

Bad example, but I hope you got my point.

* This *may* be confusing (dist assets folder, source assets folder) to some end-users.
* We are sadly lacking manpower and I don't want to push anyone with extra work since I'm also quite busy, too :disappointed:

To be honest, about this commit, I am not happy with generating a static file dynamically on each request, and give extra work to CPU and PHP, which outputs the almost same output for each individual or even HTML crawlers.

Same code can be generated with same method and can be saved into somewhere in ftp instead of showing dynamically, and update / regenerate it only when required.


`Let's require a clean source tree before running local checks to
avoid those false positives.`

It's false negatives, actually. To be picky, I'd consider fixing the commit message. Otherwise, this looks fine.


bum tralalal


It reserves the memory directly on the stack, not on the heap, which is way cheaper. So it's *not* equivalent to ```new```ing. (The elements of the ```vector``` are ```new```ed, however.)


OK, I have read the access=official wiki more thoroughly and it seems we do not need to increase priority of *=official way too much. If the bike/pedestrian is obligated to use the way, there should be foot/bicycle=use_sidepath on all parallel roads. So I make those -1 and OsmAnd should automatically use the remaining way (the one with foot/bicycle=official). This patch should be applied instead of the first 2 patches.


Compared to this govenor

https://github.com/jgcaaprom/android_kernel_oneplus_msm8974/commit/1e99d1be846f6602524bf4cb972875c8fe088f05

yours is the most updated from this project?

This govenor is very nice. You know any other with similer characteristics? thanks


I couldn't figure out a way to automatically create the database if it doesn't exist yet.


Ah, right. Actually I haven't decided on a default yet, just updated to 0.1 to make sure everything is calculated properly.


was soll das???


LGTM


`accountid` appears now twice in the query. As `SessionAccountHandler::getId()` calls `SharedLinker::getUserId()` on shared pages, only the else-case is needed.

And, `stopAddingAccountID()` is not needed as it always checks whether `accountid` is present in the query.


Oh, I suppose so. I wasn't sure. :P


Hi. I'm not familiar with what this is for. Are there other Moodle activities that have this?


Thank you very much.


This isn't needed as you can get fcut like this

 RC = 1.0f / ( 2.0f * (float)M_PI * (float) currentProfile->fcut );


This change set looks good (although I am not sure that we actually need to check the ContentLength, but it won't hurt!


Thank you.


I suppose `self.yum_base` has also to be set to `None` when we get the `NameError`.


It's ok.