Having brackets on one line if statements is the java convention. Just because they are not needed does not mean it is better not to have them. What reasons, apart from personal preference, do you have for not wanting them?

See: http://www.oracle.com/technetwork/java/javase/documentation/codeconventions-142311.html
Quote: "
Note: if statements always use braces, {}. Avoid the following error-prone form:

if (condition) //AVOID! THIS OMITS THE BRACES {}!
    statement;
"


@ohadlevy is this what you mean? https://github.com/theforeman/foreman_discovery/pull/180


Generally if you've not made any db changes i.e. added new migrations, leave the schema.rb as is. If it is marked as changed after running db:migrate, just check it out again.


Could you refactor this to use the same simple syntax you did in https://github.com/SjonHortensius/pfsense/pull/160/files#diff-5730881a53b31e3332c621a2f0a799a7L70 ? Thanks!


Hehe, was about to open a PR for this, installing 'dev-master' failed because of this


OK


Please create a proc that generates the NPC icon then replace the code for both teachers and ghosts, since it's both copy pasted code there's no reason it can't go in a proc.


I'm looking at the proto file, and everything that's "optional" can be null (unless it is overwritten in the java layer, like "date")

message PaymentDetails {
        **optional** string network = 1 [default = "main"]; // "main" or "test"
        repeated Output outputs = 2;        // Where payment should be sent
        required uint64 time = 3;           // Timestamp; when payment request created
        **optional** uint64 expires = 4;        // Timestamp; when this request should be considered invalid
        **optional** string memo = 5;           // Human-readable description of request for the customer
        **optional** string payment_url = 6;    // URL to send Payment and get PaymentACK
        **optional** bytes merchant_data = 7;   // Arbitrary data to include in the Payment message
}

message PaymentRequest {
        **optional** uint32 payment_details_version = 1 [default = 1];
        **optional** string pki_type = 2 [default = "none"];  // none / x509+sha256 / x509+sha1
        **optional** bytes pki_data = 3;                      // depends on pki_type
        required bytes serialized_payment_details = 4;    // PaymentDetails
        **optional** bytes signature = 5;                     // pki-dependent signature
}


For sure. I actually already have that, but this code part hasn't been rewritten to use it.


You have a typo here; please correct $cus**t**om_databases. Also, please reduce indenting in line 470/471 by 1 tab


Sorry, this is kind of old. I was using the repo momentaneously as an svn development repo because i have 2 different dev environments. If you start looking or contributing to the project i will take care with this.

Nevertheless, I already implemented the polymorphic visit as you suggested. I haven't commit because i still have some problems. My idea is the following:

- Create a MateNode as the wrapper. I would like this node to be a "Mate" dispatch chain with the first item in the chain being the original node as we already discussed.
- Preprocess all the AST before executing for wrapping every node with a MateNode.

I already did this. I still have some problems with types. Node childs can not be linked to my mate parent node because of type mismatch and that kind of stuff. I was already looking to the wrapping nodes of Truffle for instrumentation but i still haven't found the best solution. I think i will have to do some ad-hoc processing for linking the nodes the way i need. Hope to finish this asap.


Please note that we currently only support ACTION1 and ACTION2. The basic idea was, that ACTION1 is the primary action that person does, and ACTION2 is the second action some settlers might do (but only few have 2 actions). Having more than one action makes things complicated, because we always need to remember which is which, having 6 actions makes things more complicated.


O kodėl visur naudoji atitraukimui nuo krašto 4 tarpus, o čia - tik 2? :)


Using quotes around the L-part is not necessary, and in fact confuses one of the POD-parses (don't recall which).

E.g. the "additional DBIC resources" link [under this heading](https://metacpan.org/pod/DBIx::Class::ResultSet#FURTHER-QUESTIONS) properly points to `...DBIx::Class#GETTING-HELP-SUPPORT` while being defined as [`...|DBIx::Class/GETTING HELP/SUPPORT>`](https://github.com/dbsrgits/dbix-class/blob/current/blead/lib/DBIx/Class/ResultSet.pm#L4814)


This is the same if-statement as the block above? Why separate those?


@dragotin @ckamm Would this actually have been interesting for the issues we have with Windows connections/timeout/hibernate/whoknows?


Reminder for myself to have a look at the javscripts


I just do the subclassing for separating projects expecting and easier merging if Truffle and Mate evolve...


You would need to register HOMELESS1, HOMELESS_IDLE, ... as EAction. We would need to address this in logic first, so I recommend commenting these lines until we do decide how logic should handle this.


Atkreipk dėmesį, kad nuotraukos github repositorijoj liko ir yra viešai prieinamos per istoriją žiūrint ankstesnius commitus arba versijas.
https://help.github.com/articles/remove-sensitive-data/ - čia straipsnelis, kaip jas iš tiesų pašalint. Šis būdas turėtų būti naudojamas tik kai iš tiesų reikia pašalint kažkokius jautrius dalykus, pvz., netyčia sucommitintus passwordus ar asmenines nuotraukas. Pamėgink tai padaryt.


The podspec has been added to the CocoaPods/Specs repo


do it! :D


Indeed. I wanted to change as little as possible Truffle standard code, but in this case i think it is the best.


This is my very first comment for my new repo sdi.


@guruz Probably, but in Qt5 this is no longer in `QApplication`, but part of the platform plugin, so it's dead code. :-)


I leave this here :octopus:


Use `PointerComparison::equal()` here.


This should be a lowercase ```W``` ;)


Are you sure this clause is necessary? `put_resp_header/3` already checks that the header key is a binary.


Good thinking. These need to fly.


I think the function is a misnomer because it doesn't actually care whether the first argument represents a multiplication. Maybe just `apply_with_intermediate_storage` or similar? (Shorter names appreciated.)


I think the code would be easier to read if you replace the `return` by `else`.


Yes !! for now keep this @ 200 .. i'm testing if it can be lowered ..)


Added a comment to see if I can see it in my local repo.


install the platform lib from the build deps ppa
https://launchpad.net/~team-xbmc/+archive/ubuntu/xbmc-ppa-build-depends

On 05-05-15 14:30, EricV2 wrote:
> This breaks build using xbmc git an our ago installed in /usr/local
>
> xxx:~/local/src/kodi-platform/build->cmake ..
> CMake Error at CMakeLists.txt:11 (find_package):
> By not providing "Findplatform.cmake" in CMAKE_MODULE_PATH this project has
> asked CMake to find a package configuration file provided by "platform",
> but CMake did not find one.
>
> Could not find a package configuration file provided by "platform" with any
> of the following names:
>
> |platformConfig.cmake
> platform-config.cmake
> |
>
> Add the installation prefix of "platform" to CMAKE_PREFIX_PATH or set
> "platform_DIR" to a directory containing one of the above files. If
> "platform" provides a separate development package or SDK, be sure it has
> been installed.
>
> -- Configuring incomplete, errors occurred!
> See also
> "/home/xxxx/local/src/kodi-platform/build/CMakeFiles/CMakeOutput.log".
> See also
> "/home/xxxx/local/src/kodi-platform/build/CMakeFiles/CMakeError.log".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/kodi-platform/commit/6cfabd8b94e769a0485fd06a7e62dcdc3bee7b29#commitcomment-11043435>.
>


@mcostalba:
RootPos.captured_piece_type() is not enough because promotions are excluded.

perhaps something along this lines (that's only a rough idea out of my head, probably errornous)

RootPos.st->previous != nullptr &&  RootPos.st->materialKey != RootPos.st->previous->materialKey

@lucasart
Let's see what this gets. But one possible problem with VoyagerOne and your patch is that position analysis (with cleared hash) is not deterministic anymore.
If you load a position and analyze it, later clear hash of course and reanalyze the position you get different results for one core. Only a engine restart helps.


I'll have a look at the javascripts and possibly create an inline form for this (instead of *untitled* inputs with help-texts)


Do you mean the name of the method to be `testReplicationDocWithEmptyId` and not `tesReplicationDocWithEmptyId`?


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


`PointerComparison::equal()`.


Test failure unrelated...

```EmailIntegrationTest::testEmailImages
Facebook should return HTTP 200
Failed asserting that false is true.```


thanks!


Ehm, why do you need to subclass here? Do you access `objectClass` or other fields via MateClasses? If not, you don't need to subclass, just remove the `extends Classes` and it should still work fine, no? That's just nitpicking while I was browsing the changes, to get an impression of how you are doing things. Not essential.


Revert the schema.rb changes then you can merge to master, push then deploy to beta.


I would keep it, because it ensures that a valid header key can only be a binary, independent from guards of calling functions.


A namespace AwMath will be useful to store math functions and avoid naming confusions.


Oh I see you removed my comment proving you were incorrect about this being safe. Here's a link to a nice big post on it for anybody still looking at this.
http://forum.xda-developers.com/showpost.php?p=54960376&postcount=412


Gettext bug? I never worked with Gettext. Sorry.

Can you test with any other services? Unit test works for your other custom dependencies?


cloakedninjas-

This looks great. Would you mind adding some documentation and an example to the README?

Thanks.


Nice catch, wish I had seen this when I made the change originally ;(


Use Collections.singleton


ok... I see why there's no span reported. This error occurs during trans. There's no (easy?) way to get the ast node from the trans type.


Remove this constructor altogether. It isn't needed.


Hi Eduardo,
I have a problem. I have this numeric mask: (##) ####-####. But we are able to input special characters ( $,',").Is it normal or a bug?
My project use Vaadin 7.4.1.

Marina.


Wish I had seen this as well. I was testing your PR with ``rake provision``, which defaults to ubuntu on docker, which in turn uses upstart. Seen this one in a project where I need to deploy tomcat on CentOS with SysV. I consider this luck ;)


These functions need to move to private methods on a spline 2d object.


idem.


Ah, perfect. Yeah, that'll be a lot nicer. Thanks @juliangruber -- I'll patch today once I finish up the API samples.


Works now. thanks.


If insertionString is inherited (when the client processes the string). Shouldn't this method also take that into account?


加这个 json 文件是做什么用？


convert to class/struct will also allow for some optimizations like not recomputing the weights for new data on the same interpolation grid.


Of course, this final release is before code-review interview. this project keeps going on.


`Indices` not `Indicies` :D


Yes, having the versioned file names was pointless, it makes it very difficult to maintain and very difficult to browse, too!


Right, i understand now, all static stuff so removing the extends still
works.

2015-05-05 9:54 GMT-03:00 Stefan Marr <notifications@github.com>:

> Ehm, why do you need to subclass here? Do you access objectClass or other
> fields via MateClasses? If not, you don't need to subclass, just remove the extends
> Classes and it should still work fine, no? That's just nitpicking while I
> was browsing the changes, to get an impression of how you are doing things.
> Not essential.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/charig/TruffleMATE/commit/ca8f9c63397e52adafee34a3787851493c6a1f66#commitcomment-11043833>
> .
>


These add extra does the same as addChild. Do we need thse methods then? Should we rethink that?


This change has affected the ImageView image property which now will not display an image if there is a space in the image name.

<ImageView image="/images/thumb P6.jpg"></ImageView>

This displays the image in 4.0.0.RC but not in 4.0.0.RC2


Please note that our customizations are part of the official repo now -- https://github.com/ealdent/simple-random/pull/1. This might be a good opportunity to synch with upstream!


Why did you remove this?


This isn't symmetric.


desktop-search is important to provide proper results


Copy paste of https://msdn.microsoft.com/en-us/library/ms231597%28v=vs.110%29.aspx :) I blame MS


Thanks a lot for the quick reaction! :D


Congratz! :100:


Hah :wink:


Move this up above Convenience Mson Event. Also remove the header.


Yes, do what Chrono said .. it's really difficult to say what is the cause .. for me with the codina .. everythings works fine ... u can try: "try and error" .. just revert some commits and test .. needs a lot of time .. but u'll find the problem .. mostly of optimizations from me are doing like this ... especially the defconfig .. xD i've spended a lot of days to disable so much possible options ... and enable so options what can help for speed and powersaving ... and i've also read for every option the help text in menuconfig ... at the end i did a "end-pass" and i've sorted what should built as module .. and what should built-in ... so u guys can maybe imagining how many days i needed for this .. :-) So don't are to lazy ;-)


Compass has been in there in the beginning (and maybe never removed) but I removed all compass depended code from the sass files from day 1.

> Op 5 mei 2015, om 10:36 heeft Alexander Pilz <notifications@github.com> het volgende geschreven:
>
> Wow, nice catch. This got me good confused :)
>
> —
> Reply to this email directly or view it on GitHub.
>


Ok, what do you think if we add a AwSpline2D object with public static methods like:
spline2d()
and private ones like (find_weights, ...)


In general, have you also tested the result?
And no, I won't include specific bias for commercial sites like eBay and Amazon unless there's something on the other side of the equal sign. It's free advertising otherwise.
Also, yes there is a good reason why DDG has a pale moon indicator in there.


You should use `$fa-var-warning` instead of `\f071`, it will be easier to upgrade font awesome later ;)


Looks good to me! :+1:


Where do non-linux standalone builds get the library from? Looks like we're missing that logic in kodi right now right?


J'avais complétement zappé depuis le temps :p
C'est fait #1238


ok


Wondering if it should really log into the default channel. Even the `JsonFormatter`, but this obviously cries for an extension point.


:( msft. :(


This thing is converted to a list and then back to an array. Should we think about that for a second?


Shouldn't we have one test that asserts that `jQuery.unique` & `jQuery.uniqueSort` are identical?


That is nice. I remeber noticing that the `readyEvent` was not quite ready for primetime. The combo with `emtpy()` and `$.Callbacks('memory once')` was what I was looking for last time I worked on srcissors ;)


When fixing the code it's good to also fix the debugging information provided by `console.log`. Here you should change `*` to `+` and put the addition into parentheses.


We also need a toJson method returning a JsonElement.


from depends or the prebuilt binary on windows. did I miss one?

On 05-05-15 15:05, Sascha Montellese wrote:
> Where do non-linux standalone builds get the library from? Looks like
> we're missing that logic in kodi right now right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/kodi-platform/commit/6cfabd8b94e769a0485fd06a7e62dcdc3bee7b29#commitcomment-11044011>.
>


Ok, so static methods are not suitable.
Better implement a AwSpline class with private data and weights.

Quick interpolations will be required when the user is browsing data, and clicks to refresh the topography on a particular position.


if / else with one instruction: either write them on line or put { }


Instead of using an empty list. Couldn't we just put the for loop inside an is statement?


Dis rly cool!


protected please. Also shouldn't there be a setter for these values?


Its a bug, probably related with the use of SHIFT key. Please, fill a bug.
Maybe I can do something this night...

Eduardo Frazão

2015-05-05 9:58 GMT-03:00 marinasp <notifications@github.com>:

> Hi Eduardo,
> I have a problem. I have this numeric mask: (##) ####-####. But we are
> able to input special characters ( $,',").Is it normal or a bug?
> My project use Vaadin 7.4.1.
>
> Marina.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EduFrazao/vaadin-masked-textfield/commit/1a318e0f67ceb4b8ba2e5b946e5f933e7cdf9d45#commitcomment-11043894>
> .
>


yes


would this be ok?

tmpStr = regexp(channelLib.(strcat(genvarname('M-'),settings.Qubit)).physChan, '-', 'split');

Or do you prefer to specify the measurement pulse explicitly? The reason why I think it can be hardcoded is that names for qubits and measurement pulses are already linked in QGL


Yes, that's a good starting point.

The BIP70 support needs a bit of looking at to make sure it's robust. At
the moment we don't think it's used a great deal but that will change in
time so if you spot a potential NPE let me know so it can be fixed.

On 5 May 2015 at 13:42, Olle Kullberg <notifications@github.com> wrote:

> I'm looking at the proto file, and everything that's "optional" can be
> null (unless it is overwritten in the java layer, like "date")
>
> message PaymentDetails {
> *optional* string network = 1 [default = "main"]; // "main" or "test"
> repeated Output outputs = 2; // Where payment should be sent
> required uint64 time = 3; // Timestamp; when payment request created
> *optional* uint64 expires = 4; // Timestamp; when this request should be
> considered invalid
> *optional* string memo = 5; // Human-readable description of request for
> the customer
> *optional* string payment_url = 6; // URL to send Payment and get
> PaymentACK
> *optional* bytes merchant_data = 7; // Arbitrary data to include in the
> Payment message
> }
>
> message PaymentRequest {
> *optional* uint32 payment_details_version = 1 [default = 1];
> *optional* string pki_type = 2 [default = "none"]; // none / x509+sha256
> / x509+sha1
> *optional* bytes pki_data = 3; // depends on pki_type
> required bytes serialized_payment_details = 4; // PaymentDetails
> *optional* bytes signature = 5; // pki-dependent signature
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ollekullberg/multibit-hd-1/commit/b61e9b6289be50af6df93c6ff3c643f6412b8f4c#commitcomment-11043626>
> .
>


win32 does not use depends and the cmake buildsystem doesn't rely on prebuilt binaries anymore. And all other platforms don't use depends either anymore when using the cmake buildsystem.
But the cmake buildsystem also doesn't refer to the updated `kodi-platform` yet. It still points to 48bdd985.


Thanks :)


Isn't this already covered by the pattern `build/` under `# Gradle files`?


Thanks for your PR! Only two remarks:

# Just name it `Close()`. I personally don't like the `DoSomething()` naming :D
# There's more to do than just closing the camera: Think of the consumer of the library calling another function after calling `close()`, which could easily happen in the ansynchronous world. At least we should throw some descriptive error (for example "Calling `<method>` on already closed camera handle"). Not sure how the module would behave right now.
# Please add tests, esp. for edge cases like the above.

Thanks for contributing!


li is a pairwise tag - every open <li> should have a matching closing </li>


@locus2: that's not a problem. these days everyone uses SMP, and analysis is not deterministic already.
if you want to fix that non problem, the correct way would be to clear stats upon receiving Clear Hash or ucinewgame, definitely not after every move, because it loses elo!


I'll check in the evening. the other platforms did build libCEC fine,
which also uses this.

On 05-05-15 15:10, Sascha Montellese wrote:
> win32 does not use depends and the cmake buildsystem doesn't rely on
> prebuilt binaries anymore. And all other platforms don't use depends
> either anymore when using the cmake buildsystem.
> But the cmake buildsystem also doesn't refer to the updated
> |kodi-platform| yet. It still points to 48bdd98
> <https://github.com/xbmc/kodi-platform/commit/48bdd985b843353eec5804e4dbab2dc8f2b4b13e>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/kodi-platform/commit/6cfabd8b94e769a0485fd06a7e62dcdc3bee7b29#commitcomment-11044097>.
>


Тезисы к дипломке?


ded


btw: did u test if it'S possible with more that hz=1000 ??

Theoretically with Hz=2000 = 10ms .. = like NOHZ ---> 10ms !! xD


Yes but libCEC is not built with cmake. For the normal build of kodi this should work because you added it to depends and for win32 libplatform is included in the prebuilt binary of libCEC. But for addons which are built with cmake that isn't enough.


I know that in this case it won't raise any real issue (so it is fine) but you did not really solve the critical raised previously with this call to compare. (which could be rewritten to value.compare...


test abi_x86_64 swap


Now I see: in your code a missing "date" will be: setDate(new DateTime(null)) => setDate(null) => no exception. I'm changing to:

setDate(paymentSessionSummary.getPaymentSessionDate().orNull())

.. even though "date" can't be null with the present bitcoinj impl.


LGTM


aww, sorry, missed: https://github.com/spring/spring/blob/develop/rts/Rendering/Fonts/CFontTexture.cpp#L176

(fontFile is passed through ~3 classes...)

will revert


fixed https://github.com/MediaArea/ebml-specification/commit/f9169723bdab23ebf2a2049253160f9b27acd8df


pep8 please :)
in fact, integrate `flake8` tool into your editor, it will show you what is what.


test is incorrect.
ma2 has three entries now.


We found the issue, I couldn't reproduce it because I was using io.js.
It is fixed in es20xx: https://github.com/JSRocksHQ/slush-es20xx/commit/3b890ba2313bb1395504992cc0bf76c7672519bf
I'll check whether we can get a cleaner solution before landing in Harmonic.


well, then it'll have to be hooked up. the platform lib is built with
cmake too, so it shouldn't be more than just pointing to it.

On 05-05-15 15:15, Sascha Montellese wrote:
> Yes but libCEC is not built with cmake. For the normal build of kodi
> this should work because you added it to depends and for win32
> libplatform is included in the prebuilt binary of libCEC. But for addons
> which are built with cmake that isn't enough.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/kodi-platform/commit/6cfabd8b94e769a0485fd06a7e62dcdc3bee7b29#commitcomment-11044176>.
>


This now breaks for me 100px earlier...

Probably because you forgot to include the status in the percentage based width...

I think we wanted to give it 15% too


Thanks a lot!  :)
Fixed in b7e6ab348b2d8f4f3d694555a9802157fa2a0a1d


I don't understand why the position is moved on 8 here and only 5 in UpdateMessage.java


Hab mit ihm fuer morgen einen termin gemacht ;)

----- Reply message -----
From: "UniSabina" <notifications@github.com>
To: "Fallstudien/Optimalsteuerung" <Optimalsteuerung@noreply.github.com>
Cc: "allgaeu" <johannes.milz@tum.de>
Subject: [Optimalsteuerung] @UniSabina, @schweinekrippe, (513b9d7)
Date: Tue, May 5, 2015 12:31

ja ich würde mit dabei sein.

---
Reply to this email directly or view it on GitHub:
https://github.com/Fallstudien/Optimalsteuerung/commit/513b9d73772b01fadedceca5d157c1607f0fa4c0#commitcomment-11041852


Thanks. Looks like (a lot?) duplicated code: at least lines 156-159 and 252-254 are equal. Can't see here what is in between, but refactoring might be reasonable (at least extract method deleteReservationState(model)).


Just verified, 15% for the status (instead of 96px) fixes the issue for me.


Esto hace falta :innocent:


That is a good argument. I'm still inclined to remove it because of [YAGNI](http://en.wikipedia.org/wiki/You_aren't_gonna_need_it) tough :). We only call this from `put_resp_header/3` so we're sure we're getting a binary here. In general, this is a private function so we can control exactly what gets passed to it; we don't have be defensive.


Is there a reason this is made a user default? Since this is unlikely to be changed at any point during the app's lifetime I think it would be more suitable to make this a compile time option and not even instantiate the example label, similar to:

https://github.com/chb/AppCore/blob/master/APCAppCore/APCAppCore/UI/Onboarding/SignUp/APCEligibleViewController.m#L115


Removed inaccurate count of Element IDs in https://github.com/MediaArea/ebml-specification/commit/a6a015a5cca6b39f6d25fbd17c3598d16ed532e1 (the correct counts from the RFC draft remain)


I get that this is an error string users would normally never see because of JS validation but it is still an error string they might see. Would it be better or worse to change this into a simple error string and leave out the validation result stuff and have that simple string come from translated strings?


Weird behavior , when running MA example (in example package).


How about:
https://github.com/ClusterLabs/pacemaker/pull/700


Same (one line of { }) sorry...


I found a nasty bug related to this functionality. It comes from this code in bbPress core:
https://github.com/Skwerl/bbPress/blob/5756d5f7b31544684bded68087a4dcfcde13e921/includes/forums/functions.php#L1734-1742
If there are any private/hidden forums, that associative array is added to the `meta_query` which uses an **OR** operator. The underlying `WP_Query` object that powers bbPress pagination winds up counting posts that are not sub-replies **OR** posts that are not in hidden forums... *Any posts, from any thread.*

So the pagination gets completely thrown off (unless you're an admin, in which case, the `meta_query` isn't touched since you can see everything anyway). Topics that should have 20 pages wind up with hundreds, and all of the reply linking and post-submit redirection gets thrown off.

The upshot is that having any private/hidden forums will break pagination. For now.


Nice. :+1:


oh and btw, libCEC *is* built with cmake, but I guess you mean Kodi's
build system's cmake ;)

On 05-05-15 15:15, Sascha Montellese wrote:
> Yes but libCEC is not built with cmake. For the normal build of kodi
> this should work because you added it to depends and for win32
> libplatform is included in the prebuilt binary of libCEC. But for addons
> which are built with cmake that isn't enough.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/kodi-platform/commit/6cfabd8b94e769a0485fd06a7e62dcdc3bee7b29#commitcomment-11044176>.
>


Thanks for the fix, release 3.6.4 works for me as well!


@mcostalba
Originally that is what I wanted to do...but wasn't sure on how to clear the stats when a new game starts.  I figured my hack was a sure way to do that. (Except for rare games that engine draws without any captures)

If this pass, I except that my code would be cleaned up...by someone more knowledgeable than me.

Also my code doesn't account for promotions.  I wanted to clear the stats when there was a promotion...


This commit broke a downstream project for a short time. What's the best way for me as a maintainer to be notified of these kinds of changes? This commit doesn't seem to be related to any pull request that I can see, is there an active mailing list or is activity mainly through this repo?


Just sharing this here so I don't have to faff with FTP

![sixth](https://cloud.githubusercontent.com/assets/142650/7473382/b3797010-f332-11e4-8da9-229fba82ceab.png)


Yes, configurations like HZ=10000 are possible
https://github.com/ChronoMonochrome/Chrono_Kernel/blob/5837f2ec4130c9df4cf0e5ee3465bbd70e87f667/include/linux/jiffies.h#L34

> Theoretically with Hz=2000 = 10ms .. = like NOHZ ---> 10ms !! xD

What are you talking about? When task queue is empty, timer will not tick (so it works like NO_HZ supposed to). And if there are any active tasks, timer will tick with selected in config duration.


I'd like to point out the issue with using `-` in the **middle** of a regexp when trying to literally match `-`.

Eval the below 3 forms to get an idea of what I am trying to tell:

```el
(string-match "[+-*]" "-") ; nil
(string-match "[*-+]" "-") ; nil
(string-match "[-*+]" "-") ; 0
```

Based on http://www.gnu.org/software/emacs/manual/html_node/elisp/Regexp-Special.html, I have this rule of thumb to avoid such confusions:

- A regexp within `[` and `]` should be constructed as follows: `[<SPECIAL-CHARS1><RANGES><MISC-CHARS><SPECIAL-CHARS2>]`
 - `<SPECIAL-CHARS1>` can be `]` or `-` (but `]` must precede `-` if using both)
 - `<SPECIAL-CHARS2>` can be `-` or `^`


Fixed in translate. Will be imported with the next language update. Thanks.


thanks for fixing that :D kinda forgot about monospace typeface


@artragis : il lui manque pas ses enfants, à ce manifest ?


libplatform needs to be added to kodi-platform (cmake) depends.


specific_hits/mapping_hits    instead of    specific_hits


Looks good to me.


want to change this... let's add isosceles functionality and what's the best way to implement both things at the same time.


add cap3 and vibe motor


egg


Some food for thought.
Another idea I had is to have a reset counter.  Such as:
If stats were not clear after 5 moves, then clear the stats.


I assume because it was a fairly pointless wrapper around libevent functionality. The very few applications that need that functionality can just call bufferevent_set_rate_limit() directly.


Was it really a typo?
I think that `mimeTypes` **SHOULD BE** 'text/plain', because `FileValidator::validateValue()` checks `mimeTypes` against the result of `FileHelper::getMimeType()` which returns 'text/plain' instead of 'text/csv' in this case.


@JeromeMartinez prefers the term "element ID" as a replacement to "Class ID", but "Element ID" already has another meaning within the document.

There's also two references to "Class ID" which have different meanings. Within the "Element ID" defintiion "Class ID" is a grouping of EBML Element IDs by Width. I propose using the term "EBMLIDLength" in this section instead of "Class ID".

Later in the semantic table "Class-ID" is used to state the hexadecimal representation of the Element ID. The HTML labelled "Class-ID" as "EBML ID". I propose using "EBML ID" and removing "Class-ID" here, this would make it more similar to the Matroska specification.


Did you intend to put $name here?


That looks like a good improvement, but I would put the default costs in the config section at the beginning of the file. For some people it might be hard to find the default costs inside the function somewhere in the script (if the want to change it)!? If you add a simple config variable up there I will merge it :)


You like that? huh?


oops, typo


use a `conf` file here (see other templates.)


This code does not compile...


perhaps use a tagged version here, also for the URL below then.


use build_style=fetch.


1: M111 unrecognized component name: legacy


I'm attempting to get faster at building these.


1: D100 Missing docstring in public module
74: D400 First line should end with a period (not 'e')
74: D205 1 blank line required between summary line and description (found 0)
98: D400 First line should end with a period (not 's')
98: D205 1 blank line required between summary line and description (found 0)
118:9: E266 too many leading '#' for block comment
144:13: E266 too many leading '#' for block comment
247: D103 Missing docstring in public function


28:1: E402 module level import not at top of file
29:1: E402 module level import not at top of file
30:1: E402 module level import not at top of file
31:1: E402 module level import not at top of file
37:1: I100 Imports statements are in the wrong order. import cgi should be before import time
37:1: E402 module level import not at top of file
38:1: E402 module level import not at top of file
39:1: I100 Imports statements are in the wrong order. import binascii should be before import subprocess
39:1: E402 module level import not at top of file
40:1: I100 Imports statements are in the wrong order. import StringIO should be before import binascii
40:1: E402 module level import not at top of file
42:1: E402 module level import not at top of file
43:1: I100 Imports statements are in the wrong order. from functools should be before from flask
43:1: I201 Missing newline before sections or imports.
43:1: E402 module level import not at top of file
44:1: E402 module level import not at top of file
45:1: I201 Missing newline before sections or imports.
45:1: E402 module level import not at top of file
46:1: E402 module level import not at top of file
47:1: I100 Imports statements are in the wrong order. from urllib should be before from six.moves.urllib.parse
47:1: I201 Missing newline before sections or imports.
47:1: E402 module level import not at top of file
48:1: I100 Imports statements are in the wrong order. from itertools should be before from urllib
48:1: E402 module level import not at top of file
49:1: E402 module level import not at top of file
133:1: E402 module level import not at top of file
136:1: F401 'pyparsing' imported but unused
136:1: E402 module level import not at top of file
137:1: I100 Imports statements are in the wrong order. from flask_testing should be before import pyparsing
137:1: I201 Missing newline before sections or imports.
137:1: E402 module level import not at top of file
215: D101 Missing docstring in public class
216: D102 Missing docstring in public method
251:5: E731 do not assign a lambda expression, use a def
371:5: B901 blind except: statement
792:9: B901 blind except: statement
813:9: B901 blind except: statement
837:9: B901 blind except: statement
869:9: B901 blind except: statement
880:13: B901 blind except: statement
903:9: B901 blind except: statement
924:9: B901 blind except: statement
941:9: B901 blind except: statement
1191:9: B901 blind except: statement


This commit broke ldfReader.cpp.


it would be nice if this is configurable


You really shouldn't be storing binaries in a git repo unless all are true:

1. The code is dependent on the binaries
2. The binaries can't be compiled automatically as a dependency without including additional source code in the repo
3. The binaries are very small
4. The binaries are rarely updated

Otherwise, there are far better solutions, such as git-lfs (for frequently changing dependency binaries), GitHub Releases (for hosting build artifacts), and git submodules (for providing the source code necessary to build the binaries), just to name a few common use-cases.


use the fetch build style for this. https://github.com/voidlinux/void-packages/blob/master/Manual.md#build-style-scripts


Very nice thanks :)


0x01 Port Scanner


I messed this up. This commit actually adds my (very basic) build script. Fix commit incoming.


WSHP_SHR_GreaterThan_1 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


Awesome!


Oooh, thanks for the catch!
PR for commcare_2.21 branch : https://github.com/dimagi/commcare-odk/pull/259


Wrong place to ask for [releases](https://github.com/SecUpwN/Android-IMSI-Catcher-Detector/releases), @emeries13. In the future, contact me via E-Mail.


The parameter is just fine but the description is wrong. :smile:  It should return all specified attributes, not non-empty.


I think this should be split into two commits.


Yes, because the site is statically generated.


Agreed, if we don't have a strong case for using it for anything else I'll just include this function. :+1:


ahh yes.. i've again meesed it up with governors^^ 10ms is for cpufreq.

so i need to apply that commit ?? or u just showed me .. ?

but hmm 10000hz .. but what is the limit or better: how much hz is need for "real-time" ??? And what about overhead ?? Not that we're decreasing speed xD

ok lets calculate something...

HZ=10000 = 2ms ... HZ=20000 = 1ms ... lol  and HZ=40000 = 0,5ms  HZ= 80000 = 0,025ms .. so i think this is fast enough xD


@rodrigopavezi I've been thinking about this and I can't figure out whether or not it will cause issues going forward. Why did you move this event out of `_authorize` to `authenticate`? Authorize is called internally (on page load and re-focus), and if we do this, the event is not going to be called in those instances. I don't think that's desirable; I think it needs to be moved back. @olegrise thoughts?

Thanks.


this was a tricky problem, I was doing gymnastics to get this to work the way I wanted. I'll ping you on vc to explain all the things I tried and see if you have any ideas.


Thanks for all the notes, going to ping you when I make the pull to review and merge if you don't mind? Really appreciate the feedback. :+1:


@alex-deaconu @rodrigopavezi I had the same questions.


WSHP_SHR_GreaterThan_1 (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@locutus2 don't considering promotions should have zero impact, eventually we could run the captured_piece_type version with a SPRT[-3, 1], although it would seem and overkill.

@VoyagerOne ditto for not clearing at games start: zero impact !

Please don't wait for someone to cleanup your code: I strongly recommend you clean up yourself. Thank you very much!


We should not be making commits to a release branch without even having a testing platform.  I am trying to work with another user to fix skin issues with 1.12 and this is blocking my work.  I would like to revert this to fix the build until we have a more stable fix.


WSHP_SHR_GreaterThan_1 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


This useless code is removed in 1c354ca


(lol)


Is this special char was added on purpose? :)


WSHP_SHR_GreaterThan_1 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

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


The class `APCDemographicUploader` is not in the repo. Also, I think the `configureObserverQueries` should be moved to the user-did-consent notification captured in the app delegate.


I would move this to a helper method. what do you think?


As a general guideline: in case of zero ELO impact, please always go with the simplest and clearest version.


csv MIME-type is 'text/csv'. https://en.wikipedia.org/wiki/Internet_media_type#List_of_common_media_types and https://github.com/yiijan/yii2/blob/master/framework/helpers/mimeTypes.php#L147

@samdark Did I not understand something?


The TYPEOF -> TYPE-OF change runs afoul of package locks on CL:TYPE-OF. I can't build cl-rethinkdb as a result.


I think it is `AttributeError`


пункты по которым мне нужно сделать текстовый вариант дипломки .


Thanks for the recommendations. But my other project - Xenos depends directly on compiled binaries. And compilation of these particular binaries requires installation of additional build tools that are not required to build Xenos itself.
I'm currently thinking of moving binaries to git-lfs though.


Because of the way the error handling mechanism works, you don't have to put an if (err) return err after each check. The errors will "fall through" if one validateParameter:-style call fails.

You can remove all the "if (err) return err" statements except the ones at on line 178 and 185, and it will work fine (and be a little more readable).


> so i need to apply that commit ?? or u just showed me .. ?

No, you don't need to. I reverted that commit as it caused build errors when I tried to change HZ.

> but hmm 10000hz .. but what is the limit or better: how much hz is need for "real-time" ??? And what about overhead ?? Not that we're decreasing speed xD

It's just for reference... surely I never mean to try use HZ=10000 on mobile :)

How do you calculate it?
Isn't it should be so(just an example):
`ν = 1000 Hz`
`t = 1 / v = 1 ms`


:-(

Internal Server Error

The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

Previous build works for me


This closes #121


:+1: :100:


take this code and put the delay in this line. this should have a better performance.


Please change the design of this function so that it takes three parameters like this:

^subarray($cities|1|2) instead of ^subarray($cities|^array(1|2))

Using the inner ^array() for grouping isn't necessary because the parameters can be specified individually with a little less typing.


this is a public commit, I can still see your password (again)


@danielgomezrico be more careful,please.

-* [Gesture](#Gesture)
+* [Gesture](#Gesture)g


Can you update the comment as well?


qqq


Please make the changes noted, and I will merge.


I pointed it out and didnt get a thanks, but hey I'll point it out again, you re-commited the BACKUP of your DB password file in your commit again.

 include/mysql_connect.php~ (still contains your password!)

you should review your commits before you just commit EVERY FILE. break them up into logical commits instead of one giant commit.


Hi, there is no problem with it, these credentials are unused, just there for example.
But thanks anyway.

Le 5 mai 2015 15:49:21 CEST, Stu <notifications@github.com> a écrit :
>this is a public commit, I can still see your password (again)
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/CaptainStouf/ecDB/commit/c8b83174c757ada54248ba0b16b4ebfecf72e484#commitcomment-11044815

--
Envoyé de mon téléphone Android avec K-9 Mail. Excusez la brièveté.


I like the thought process behind this but I think both mine and your versions aren't quite good.

Maybe it would be good to change to:
Let's assume we've added in another property to adjust the css left attribute


It was the first thing I done. But then I run across more similar stuff and I wanted to discuss it with you. I put down some notes so I will include them in next commit.


dunno.. but according to this example:

HZ=1000: min=20000us (20ms)
HZ=250: min=80000us (80ms)
HZ=100: min=200000us (200ms)

so HZ=10000 = 2ms and so on ....

My calculation is right .. or not ?


@Syssx 230 already here


"legacy" :smile:


Hmm, I take it you’re using stubby as a module (as opposed to from the CLI)? I’m not too familiar with the code beyond what’s in my pull request, but from skimming the prototype, it looks like maybe adding an `body instanceof Buffer` condition to `purifyBody()` (endpoint.js) would do what you want? I haven’t tried this, though.


@softark
fileinfo defines `csv` as 'text/plain'?


Thats good to know mate! I did not want to see you have trouble with posting a password.


can you give link, as I'm not well understand what does mean "min".


IIUIC, `item` is a `_PersistMember`, `entry` is e.g. `'name'` and `value` is an ID... So, there is probably a mistake.


WSHP_SHR_GreaterThan_1 (Nigusse) - Win64-Windows-7-VisualStudio-12: Build Failed

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


This was a mistake too. Don't know why these lines got busted.


@sgrif  It breaks all our `composed_of`'s. With this implementation, you completely neglect the `converter` option. And how do you ensure the ordering of the elements, if you just call the `*part.values` on the hash? We e.g. parse our api objects and I can not guarantee the proper order of the values in the parsed hash. It is up to the parsing library I use.


Would be good to explicitly set rasterDoc to 0 if img.rasterDoc was 0?  I don't understand in which situations this is supposed to be and in which it isn't.


Yes, as I mentioned, I turned "Thanks" off for now.


do we still want to linkify a source repo that is not on github?


Strange line ending replaced full files.


ses enfants? tu veux dire "mon test est pas assez gros", malheureusement,
oui, je sais, je manque de temps et en plus j'ai déjà dû corriger un bug
dans la commande alors je n'ai pas pu aller plus loin.

2015-05-05 15:29 GMT+02:00 Pierre Beaujean <notifications@github.com>:

> @artragis <https://github.com/artragis> : il lui manque pas ses enfants,
> à ce manifest ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/artragis/zds-site/commit/c0c7de082fcbe90796a1ef55331f53796410dd02#commitcomment-11044435>
> .
>


Both runtime.GOROOT() and 'go env' give wrong info. Why do you propose to use them?...


Was there a reason to switch the format of the diff output?


probably that it'll tick with minimum 20ms and so on .....but not sure.. here u're again:

https://github.com/TeamCanjica/Samsung_STE_Kernel/blob/ada5cb730330749534576adcaad0df8ac9062e5a/Documentation/cpu-freq/governors.txt#L138


Feel free.


this should do it for now: https://github.com/xbmc/xbmc/pull/7069


No, it's key error since `PersistMember`s  attributes are implemented but accompanying dictionary doesn't contain requested key.

```
    @property
    def name(self):
        return self.param_dct['name']
```


Sincere apology. But just learning some git and kernel stuff. Not releasing any custom kernel using your commit. If I plan to release something in future with your commits it will be cherry picked from your repo with proper credits .


WSHP_SHR_GreaterThan_1 (Nigusse) - i386-Windows-7-VisualStudio-12: Build Failed

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


thanks

On 05-05-15 15:57, Wolfgang Schupp wrote:
> this should do it for now: xbmc/xbmc#7069
> <https://github.com/xbmc/xbmc/pull/7069>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/kodi-platform/commit/6cfabd8b94e769a0485fd06a7e62dcdc3bee7b29#commitcomment-11044998>.
>


in sites.yml file:
<pre>goodcity:
  updateUserUrl: http://localhost:3000/api/v1/users/:id </pre>


@githubjeka
As you see in the document just above the example code:
> For example, `csv` files are detected as `text/plain` instead of `text/csv`.
That's why you have to set `checkExtensionByMimeType` to false, since the extension ".csv" will not match  the MIME-type "text/palin" detected by fileinfo.


Yes it can't hurt


We need to check :)


Uh, OK...


Thanks

Dan Williams, SIQ Agile Coach
CSM, CSP, PMP
Cell: 425-877-6333
________________________________
From: Jean-Denis Vauguet <notifications@github.com>
Sent: Monday, May 4, 2015 1:09 PM
To: danielwilliams12/new_options
Cc: Dan Williams
Subject: Re: [new_options] Added two scopes for reordering queries in the post model (74bf2f0)

In the blocks ({ ... }) that define the scopes, you don't need to specify posts. as a context for the model's attributes. It means you can simplify the code so it looks like this:

scope :ordered_by_title, -> { order('title ASC') }
scope :ordered_by_reverse_created_at, -> { order('created_at ASC') }

-
Reply to this email directly or view it on GitHub<https://github.com/danielwilliams12/new_options/commit/74bf2f037fc1acbbbc045e62e793e4a0e04ce244#commitcomment-11032487>.


teste


"We apologize, but the page you requested is currently unavailable."

I'll try later

On Tue, Apr 28, 2015 at 11:49 PM, Salvatore LaMendola <
notifications@github.com> wrote:

> If you haven't already signed up, you can follow this link:
>
> https://www.transifex.com/signup/?join_project=k9mail
>
> If you're already a member, just go to the project page and look for the
> "Help translate K-9 Mail" button:
>
> https://www.transifex.com/projects/p/k9mail/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/k9mail/k-9/commit/05934d75d84f9b80ebfc716ca3b7275ff8dac5c3#commitcomment-10952168>
> .
>


I was thinking in case we ever want to switch back, or re-use any of the stuff on another gh-pages based site.


Hey @heigler  I guess there is a cyclic dependency over the mixins and helpers.
I cannot import the helpers.get_is_content_owner

/home/roger/workspacepy/myprojects/nswmicrosite_development/src/django-magic-content/magiccontent/helpers.py in <module>()
     11
     12 from .models import BaseContent
---> 13 from .views import ListContentMixin
     14
     15

/home/roger/workspacepy/myprojects/nswmicrosite_development/src/django-magic-content/magiccontent/views.py in <module>()
     11 from rest_framework.response import Response
     12
---> 13 from .mixins import (EditableMixin, CreateContentMixin,
     14                      ListContentMixin, SimpleContentMixin, LongContentMixin,
     15                      PageLinkMixin, IconContentMixin, BackgroundAreaMixin)

/home/roger/workspacepy/myprojects/nswmicrosite_development/src/django-magic-content/magiccontent/mixins.py in <module>()
     15 from .forms import (SimpleContentForm, LongContentForm, PageLinkForm,
     16                     IconContentForm, BackgroundAreaForm)
---> 17 from .helpers import get_is_content_owner
     18
     19

ImportError: cannot import name get_is_content_owner


Good catch, fixed 8701dc189801691a0112f59bba7f5d27cb4d1e76


Hey Dejan, why are we putting documentation here?  Why don't we move what is in confluence to the wiki in Github?  Hope you are well!

Brian


Could you provide some examples or tests even of the difference between these two? For documentation.


It must be a hard job!!!


So, we just didn't understand each other. So this is about show_sampling_rate_min, and I don't know, should we interprete example like you said before, or it's just a presets.


This causes multiple lookups of the object on the server to happen in a single command - I'd prefer that we not do this, but instead provide `f.LabelsForObject(mapping, object)` once the object has been loaded.  Going to the server multiple times raises the possibility someone else changes it in between.


thanks @johndeng #TIL


bug hunt*


why is this needed ?  the packages are not in RDEPENDS for packagegroup-base.


Ah, one caveat to switching to select() - you'll want to nil-check the return, since it will happily give you a nil if that was one of the arguments passed, whereas {...} omits them.


Just want to say that this tile works fine with play music, but doesn't play well with Eleven if you use it

just wanted to give you heads up


`components/episode/default.htm → components/partials/episode.htm`
`components/episode/player.htm → components/partials/player.htm`

Breaks episode component on my installation


Fixes #16
Will always place any arguments/sample_options immediately following the sample.


Nice !


yes.


really don't know...  so let's forget about this.. i think HZ=1000 is good enough :-)

But HZ=100 was definitely wrong .. because our Phone is not a Server with 10000 CPU's xD


It looks like those will always be null. Maybe you want them static?


You don't seem to be SHGetFolderPathPtr anywhere, could you remove it?


Just nitpicking, but using lowercase for the first letter of variable names and also putting a space before the '=' would be more appropriate.


can you please add me to these reviews?


I think passing nil should result in an error instead of omission. I could do an assert on all arguments passed. In general, the entire library could with some parameter checking.


Looks fine to me.  I'm pretty sure that, at least on CCL, we could just drop the special case for `(integerp low)` and `(integerp high)` because the compiler should do the constant-folding for us.  On the other hand I can't see that it hurts anything to leave it in, so I'm fine with it either way.  Thanks.


Armée


That's probably supposed to by `https://`


Thank you. Actually I'm wondering how to cover goto_end_loop_next in rspec...


Good point.


Ontbrekende spatie.


Thanks, it's now in the testing pipeline.


Please remove **sudo** from this line. **No one should ever ```sudo pip install``` anything.** But explaining one of the right ways to do it is too much for this howto. People who run into an install error when typing ```python setup.py install``` should be forced to figure out what's the right way for them to do it.


> Am I right in thinking that this line is doing the same thing as these line (in #markers & #exact_markers)
Yes, we could update those as well.

>What happens if there is a : within the expected content?

The second argument in `split` limits the times it splits the string, so the maximum of 2 strings will be returned, the rest of the `:`s will be ignored


The thing is that we never load the input object. This pattern is followed in all other Factory methods...


I agree. But if the fileinfo returns the MIME-type of 'text/csv' for csv files, then we have to rewrite the whole paragraph and the example code, replacing 'text/csv' with something else.

BTW, I think the description of the "Note" paragraph here needs a bit of improvement to clarify the reasoning.


Hi Simon!
Thanks for the PR. Only one tiny issue. I'll merge it when it's fixed.
Thanks,
Andreas


The spec looked good! Well done!  I couldn't find any examples on line that were remotely workable. :)


test second


There are a few other parts of the build which mean we couldn't switch back so this seems like an unnecessary constraint. If we think that's a likely thing, we should use the 'github-pages' gem which requires the right version of jekyll and the other GitHub deps.

With Travis/whatever making CI deploys free and easy we may as well make the most of the extra stability/features of the newer releases and remove the burden of tracking the GitHub Pages jekyll versions.


That is exactly what I am doing. I am moving all documentation from curent wiki into the github.
I am using ReStructuredText format which can be converted into various other formats, HTML in the first place.
Content of this folder is automatically pulled and build into nicely formatted documentation at https://n2cms.readthedocs.org/en/latest/


You can use attr_reader to achieve the same.


Ontbrekende spatie (2x).


1640-AddHVACPart1 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


1. Head to agile toolkit.org and download ZIP from the main site
2. Open it locally, set it up
3. Gradually copy the lost-and-found code into admin/*
4. If you run into problem, let me know.

Version 4.3 has a different project layout, it has "public" folder for assets. I'm sure this project can be also updated to work with 4.3 with some minor modifications.

Read http://book.agiletoolkit.org/controller/pathfinder.html to understand how Agile Toolkit searches for JS files and why those two folders are being searched. You might need create your Lostandfound/atk4 folder and copy content from atk4/public/atk4.


u


Massa Gustavo!


Thanks @richard42! I would test Windows first and, if it work well, test on OSX.


Maybe I'm misunderstanding the intention but I don't think that this changes anything, QDir::exists() can still return false for that path.

http://doc.qt.io/qt-5/qdir.html#mkpath would probably be necessary to achieve what you propose in the commit message.


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Yes, I agree. I believe it was default value for a long time, so nobody even admit it should something else :)


Nothing works now. Could you fix TLU before "cleanup"?


1640-AddHVACPart1 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


I am about

> Am 05.05.2015 um 16:15 schrieb Tomasz Hemperek <notifications@github.com>:
>
> Nothing works now. Could you fix TLU before "cleanup"?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/SiLab-Bonn/basil/commit/906bfa4e953dac0e9df87c4b32699368a3d14597#commitcomment-11045345>.
>


Awesome! It'd be great if a new version of rust-ascii was released to include this change


Also, GtHub supports restructured text and knows how to render it, e.g. https://github.com/n2cms/n2cms/blob/master/docs/documentation/getting-started/server-requirements.rst


Or did you mean that this adds trailing slashes?


86019526_IgNullChoice (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

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


AFAIK you can only call new(params)


Wenn eine Person mehrere Rollen hat, dann hat es die Rollen bisher in separaten Spalten angezeigt, was dann im CSV alles verschoben hat, so dass dann gewisse daten in den falschen Spalten angezeigt wurden.

Dies wurde nun angepasst, dass wenn eine Person mehrere ROllen hat, dass diese in der selben Spalte durch ein "," getrennt angezeigt werden. Jedoch gibt es Probleme mit  der Formatierung.

Sollen wir diese Änderung beibehalten, oder verwerfen? ( Gerhört nicht zur user story).


Is this reference to an external website really expected ?


STOP MESSING, packagegroup-base-smbfs is not empty, it contains requires FILE SYSTEM packages of which CIF IS NEEDED.

```
SUMMARY_packagegroup-base-smbfs = "SMB network filesystem support"
RRECOMMENDS_packagegroup-base-smbfs = "\
    kernel-module-cifs \
    kernel-module-smbfs"
```

keep this and i will ban you from oe-alliance and yes i can as i am the OWNER.


Thanks!


I also manually removed the libg2c.so link made by sherpa and replaced with links to the libs from the libg2c_bin packaged.  The sherpaX cfg should get a tweak for this.  Retesting starcheck locally and remotely.  sherpa should also be tested.


why ‘Toolbar = function Toolbar()‘?


86019526_IgNullChoice (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Actually it was already in the previous version, just in another file.


Thanks. It's the original progressbar from googlecode. I requested to get it updated to another fork, yet it would be nice to be able to run it with the current Fedora too. At least to have a workaround until it is updated properly over here.
It seems to work fine with `-n` over here :)


thanks

Dan Williams, SIQ Agile Coach
CSM, CSP, PMP
Cell: 425-877-6333
________________________________
From: Jean-Denis Vauguet <notifications@github.com>
Sent: Monday, May 4, 2015 10:31 AM
To: danielwilliams12/new_options
Cc: Dan Williams
Subject: Re: [new_options] Updated associations between posts and users (03a9590)

I edited my message, so not every parts were mailed to you. I'll just paste the full message again:

________________________________

You could add /spring to your .gitignore file.

Edit: oh, it's already in the .gitignore. Because the spring directory was already tracked though, you would have to tell git to stop tracking it:

git rm -r --cached spring

-
Reply to this email directly or view it on GitHub<https://github.com/danielwilliams12/new_options/commit/03a95901405532aace03c7c183423ad5fb21fdda#commitcomment-11029562>.


Nice!  I really like that.  Keep going!


Hm, I don't think we should copy random headers around. Mixxx should build with an off-the-shelf Windows SDK instal. Also the problem isn't that VersionHelpers.h isn't available -- there are some undefined constants.

Also, why is any console code running at all in non DEBUGCONSOLE mode?

> Maybe this fixes also the memory leaking crash.

Which crash? And what would console font changes have to do with a memory leak?

```
c:\program files (x86)\microsoft sdks\windows\v7.1a\include\sal_supp.h(57) : warning C4005: '__useHeader' : macro redefinition
        C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\INCLUDE\sal.h(2886) : see previous definition of '__useHeader'
c:\program files (x86)\microsoft sdks\windows\v7.1a\include\specstrings_supp.h(77) : warning C4005: '__on_failure' : macro redefinition
        C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\INCLUDE\sal.h(2896) : see previous definition of '__on_failure'
C:\Program Files (x86)\Windows Kits\8.1\include\um\VersionHelpers.h(113) : error C2065: '_WIN32_WINNT_WIN8' : undeclared identifier
C:\Program Files (x86)\Windows Kits\8.1\include\um\VersionHelpers.h(113) : error C2660: 'IsWindowsVersionOrGreater' : function does not take 2 arguments
C:\Program Files (x86)\Windows Kits\8.1\include\um\VersionHelpers.h(119) : error C2065: '_WIN32_WINNT_WINBLUE' : undeclared identifier
C:\Program Files (x86)\Windows Kits\8.1\include\um\VersionHelpers.h(119) : error C2660: 'IsWindowsVersionOrGreater' : function does not take 2 arguments
```


Return doesn't require parentheses in C or C++ either.


Could this be because the ```getablenamefromfield()``` is defined after being called which is not very good practice?
It could be harmlessly moved to the actions section anyway.


That's ok but please move it to separate file :)


Yes, adds all the trailing slashes required.

I don't want to create anything, the path should already exist.


Thanks


Could you make this a pull request on our upstream? Thanks


comment test


Seems like too many ways to configure the cache. I think it'd be better to use the alias idea, but default it to `None` in development.


comment line


It's a typo, i'll fix.


nice nice!


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I added it to setup.py's install_requires so if you installed shedskin via 'pip install git+ssh://github.com/shedskin/shedskin' it'll pull in the right dependencies


With the code:

        $personalizationParams = array_merge(
            $personalizationParams,
            array(
                'baseUrl' => $this->baseUrl,
                'translate' => $this->getDI()->get('translate')
            )
        );

I get the following error while using {{ translate.t('Hello') }} inside my
volt template:

    func_get_arg():  Argument 2 not passed to function

But if i write:

    {{ translate }}

I get the following error:

    Object of class Phalcon\Translate\Adapter\Gettext could not be
converted to string

2015-05-05 14:55 GMT+02:00 Serghei Iakovlev <notifications@github.com>:

> Gettext bug? I never worked with Gettext. Sorry.
>
> Can you test with any other services? Unit test works for your other
> custom dependencies?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sergeyklay/phalcon-component-example/commit/9ad7c50375ef44ad59ec16759f50ac3c7de3ec73#commitcomment-11043855>
> .
>


Il manque la classe RouteMembershipProcessor !


Armchair coding, but copy-paste bug?  `A.s-id-var` here, maybe?


nahh its fine bro..just happy to see its being used by somebody.. :+1:


PodSelectorForResource, PortsForResource, and LabelsForResource all repeatedly load the same object, but there's no guarantee they see the same one.  That's just a bad design.  I'd like to see that fixed properly.


Can @frank-trampe or some else confirm that this doesn't fuck anything up?
This is part of pr #2253


final WebClient webClient = new WebClient(BrowserVersion.INTERNET_EXPLORER_11);
        final HtmlPage page = webClient.getPage((new URL("file://" + outFile.getAbsolutePath())).toExternalForm());
        String content = page.asXml();
        System.out.println(content);
        FileUtils.writeStringToFile(outFile, content);


Hi @catsby at the moment activity notification is generally done through the repo since the SDK is not stable yet. We will do PRs for large changes in order to get feedback from the community (e.g. paginators and waiters).

At the movement the best way to be notified of changes is through the SDK's commit history. This is definitely less than optimal, and can definitely be improved. I like the idea of a mailing list or group for notifying the community of changes like this.


Please throw it all away and use this:
http://www.crosson.org/2013/07/serialization-example-using-json4s.html


+1, not tested.


Shouldn't the value returned by new FileTypeDetector().detectAndPostToBlackboard(aFile) be assigned to detectedFormat?


l9, 10 might be explicitly put in line 11 :)


Андрей, давай сделаем название одинаковое с репозиторием
SuperSelect - очень пафосно :)


Maybe rename "*One" to "*Any"? That's the lingo I know from functional languages.


thanks!  (-:


Hey  Cyrille,

I know this is used to create a new socket connections every time JMX posts to graphite.  But it doesn't override the JVM's default of DNS Caching.  For Java 7 and below it seems these DNS resolutions are cached indefinitely. This led to a pretty bad situation when we migrated graphite and changed the CNAME to the new graphite relays.  Granted this is a unique specific problem I'm wondering if there are anything we can do to PR back.  Obviously we wouldn't want to set a JVM level metric that will effect the actual apps settings.

Or am I missing something and we are somehow forcing the DNS lookup every time?


А если не будет задан текущий?


So generally you can have something like that :

```ruby
def convert(money, currency)
  exchange = "#{input_currency}_#{output_currency}"
  money.amount * EXCHANGE_RATES["#{exchange}"]
```


@fritx :beer: :beer:


Added the `PUBLIC` keyword to all `target_link_libraries` calls because that seems to be the modern CMake way.


require whole application in spec_helper :)


I was having trouble with 'scope' and I just read again the rubyonrails.org listing on scopes.

They are class methods which don't look like class methods.

The class method would look like:

class Shirt < ActiveRecord::Base
  def self.red
    where(color: 'red')
  end
end

 The scope is 'syntactic sugar' and looks like this

class Shirt < ActiveRecord::Base
  scope :red, -> { where(color: 'red') }
  scope :dry_clean_only, -> { joins(:washing_instructions).where('washing_instructions.dry_clean_only = ?', true) }
end

What it evaluates to is this:
The above calls to scope define class methods Shirt.red and Shirt.dry_clean_only. Shirt.red, in effect, represents the query Shirt.where(color: 'red').
http://api.rubyonrails.org/classes/ActiveRecord/Scoping/Named/ClassMethods.html#method-i-scope

Dan Williams, SIQ Agile Coach
CSM, CSP, PMP
Cell: 425-877-6333
________________________________
From: Jean-Denis Vauguet <notifications@github.com>
Sent: Monday, May 4, 2015 1:09 PM
To: danielwilliams12/new_options
Cc: Dan Williams
Subject: Re: [new_options] Added two scopes for reordering queries in the post model (74bf2f0)

In the blocks ({ ... }) that define the scopes, you don't need to specify posts. as a context for the model's attributes. It means you can simplify the code so it looks like this:

scope :ordered_by_title, -> { order('title ASC') }
scope :ordered_by_reverse_created_at, -> { order('created_at ASC') }

-
Reply to this email directly or view it on GitHub<https://github.com/danielwilliams12/new_options/commit/74bf2f037fc1acbbbc045e62e793e4a0e04ce244#commitcomment-11032487>.


@dwainm I'd suggest including the site name here, as they may have multiple sites. So:

"New course created on [Site Name]."


Получается у тебя создаются копии элементов во время фильтрации?


Better change this to a nicer foreach loop. e.g.:
```java
for (Tweet tweet : tweets) {
    if (!db.containsTweet(tweet.getId())) {
        db.insertTweet(tweet.getText(), tweet.getId());
    }
}
```
Intention clearer, less error-prone (no off-by-one errors).


You don't need to call half of these 'self'


shoudn't this point to the new UserStories activity?


@bfcapell, were you successfull with all those changes?


@dwainm I'm not sure 'Edit Assigned Course' makes sense. How about:

"You can view/edit the course here: [Course name]"


I wasn't sure why we are linking to QT_LIBRARIES here directly. The call to `QT_ADD_MODULES` later within this script should take care of that, no?


@ihodes FYI for ES6 classes and React. We might want to do this for CycleDash; it seems to be the way React is moving.


What if both currencies are missing ? ;)


This was also added in #1069


Include it in your Exchange class or move it to separate file.


I didn't get into the event stuff. I meant to finish the tests so this branch could be merged into master, but I went and had my first child instead, so extra time has been few and far between.


@kislakiruben: this could just be an anonymous function, but IMO this syntax makes it more obvious that this code builds a constructor for the toolbar.


As I can see you do not use $tracks anywhere outside "if" control statement, why you need to define it?


The macro `qt5_use_modules` is deprecated, according to the Qt docs, and should only be used with some older CMake versions. So I have replaced it with the loop below that adds the link libraries and include dirs.


This script is no longer necessary.


Any reason for not using sessionFactory.getCurrentSession()?


커밋 전체에 대해서 얘기할 수 있습니다.


minor styling nit: please enclose even single line statements in curly braces. we have stuck to that convention in the rest of the code (for better or worse :))


Many things here are copied from the old FindezEngine.cmake script but is not a package config file.


Fixes #335


thanks for the PR. just have one style comment. will merge when fixed.


This looks like a typo and should probably be page.switchToParentFrame() instead?


This script is only used at configure-time when the user is including the installed version of the ezEngine. During the ezEngine configure-phase, this script will be populated with `add_library` and `add_executable` calls, such as `add_library(ezFoundation SHARED IMPORTED`, which will be executed when the user includes the installed ezEngine package.


Did you intentionally add the comma after asc?


Had to extend this script a bit to initialize and finalize some of the install-related stuff.


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


Ha, I didn't realize anybody was paying attention. That was a pretty bad commit message :/

We're not using Bedrock completely but I really like a lot of aspects of it, especially trying to treat WordPress like a 12-factor app. Your script is super helpful — I've been pulling my hair out for months over having to commit our compiled CSS assets to our repos, which completely breaks our pull request workflow for WordPress projects. This script + codeship lets us use an automated build, test, and deploy pipeline with WP Engine, so thank you!


This breaks the Invitation Application which needs the query string from the registration URL (as shown by the failing functional tests).


NOW that you mixed up server and client again it doesn't make sense anymore, that's right.


There are some formatting changes in this file that could be moved to the previous commit


Also, the changes in the HEAD linking were not required, but I figured if there were any more rsync's they would break GRETA dev if the linking didn't match.


You think? I figure with an alias, if you test one, you test the other too.


<script "grav"


@NathanW2 @nyalldawson any comment on the new implementation? I used a dock widget, so it doesn't go large, but still has the flexibility to be moved around.


Is this the correct way to add a personality file to the acceptance tests?

This file should work for both Linux and WIndows servers, the file won't be created on Windows but I don't think the Openstack API would error.


it is called absolute URL, I believe


라인에 대해서 리뷰를 할 수 있습니다.


I agree and will work on changing all those but I prefer to do it in a follow-up PR if thats' ok with you.


Does it create new array for each object?


Odd that the std logger package doesn't expose the default logger. Anyway, this is neat but they do declare they're default logger as outputting to `Stderr`:

```go
var std = New(os.Stderr, "", LstdFlags)
```

This doesn't matter/warrant a change, it's more just a heads up.


Good catch on the indentation.


Supposed to be ArrayMap as well?


Nice.

![](http://i2.kym-cdn.com/photos/images/newsfeed/000/640/962/d91.gif)


Ok, it works. Thanks


linky only supports http/https/ftp/mailto and plain email address links [1]
First I thought, that regular git addresses will be ignored and that would mean we're clear. However, right now I realized that https links might still pose a problem. With github and bitbucket, the https url will work for both git clone and regular browsing. I don't know about other providers, but I think these are the biggest ones.

I can fix this by moving the linkify code into the githubLink() filter. Should I do it?

[1] https://docs.angularjs.org/api/ngSanitize/filter/linky


I'm confused, what good is it that this is dynamic if it's not exported?


:nail_care: `If` -> `if`


Probably should get rid of this part, wasn't supposed to be here, haha.


above echo is proxy_1, why here is proxy_99?


+1 by @elikem


We could just use the logger from `config` internally, right? Wondering why the explicit parameter here.


Any ETA for this? :)


this is a test branch dont forget , then issue found all can be fixed , Please no dispute


Looks legit :+1:


I'm not sure I understand question 1.

You are correct as I spelled it incorrectly a few times.  This should be fixed in the next commit.

I have also run into the issue of these two domains showing up at the top of the list after sorting and removing dupes:
```
theoads.com.
TRIANGLESERVICESLTD.COM
```
I added them to my `whitelist.txt` and they get removed fine.

I think once the list gets to a certain size performance really suffers.  I was thinking of leaving off the mahakla list as I tend to get errors during testing.


Nothing really important behind that, just that the unified diff format is generally preferred and more readable.


i recompiled and it works now for some reason... nevermind this.


ideas welcome , Please no dispute


I believe @mzgol is saying that `strictEqual( jQuery.unique, jQuery.uniqueSort, "jQuery.unique() is an alias to jQuery.uniqueSort()" )` should be added.


I create something in the README file


Such an honor. Thanks!


thank you for saving the world


This won't be any better when the trash mailbox doesn't exist yet. The best thing to do would be to keep track of the ID in a separate variable (that is set to 'Trash' if no mailbox was found).


As far as I know you should have implemented #to_eur etc methods.


There is a patch, which I already asked for, talking about it.

https://github.com/phalcon/cphalcon/pull/10088

https://github.com/phalcon/cphalcon/commit/39a399cc26b41439512591c0bd91efebde09263d.patch

So, finally I crashed with the same stone :-(

2015-05-05 16:23 GMT+02:00 pablo moreno <pablomoreno61@gmail.com>:

> With the code:
>
>         $personalizationParams = array_merge(
>             $personalizationParams,
>             array(
>                 'baseUrl' => $this->baseUrl,
>                 'translate' => $this->getDI()->get('translate')
>             )
>         );
>
> I get the following error while using {{ translate.t('Hello') }} inside my
> volt template:
>
>     func_get_arg():  Argument 2 not passed to function
>
> But if i write:
>
>     {{ translate }}
>
> I get the following error:
>
>     Object of class Phalcon\Translate\Adapter\Gettext could not be
> converted to string
>
>
>
> 2015-05-05 14:55 GMT+02:00 Serghei Iakovlev <notifications@github.com>:
>
>> Gettext bug? I never worked with Gettext. Sorry.
>>
>> Can you test with any other services? Unit test works for your other
>> custom dependencies?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/sergeyklay/phalcon-component-example/commit/9ad7c50375ef44ad59ec16759f50ac3c7de3ec73#commitcomment-11043855>
>> .
>>
>
>


Thanks for the photo!


I like the improvements.


Looks fine, I tested staging with an in-private and it seemed to work fine. So hopefully this will be enough.


Found BROKEN by quick.
cies of target casadi
[ 43%] [ 43%] [ 43%] Building CXX object casadi/core/CMakeFiles/casadi.dir/casadi_instantiations.cpp.o
Building CXX object casadi/core/CMakeFiles/casadi.dir/shared_object.cpp.o
Building CXX object casadi/core/CMakeFiles/casadi.dir/__/__/casadi_meta.cpp.o
[ 44%] Building CXX object casadi/core/CMakeFiles/casadi.dir/weak_ref.cpp.o
[ 44%] Building CXX object casadi/core/CMakeFiles/casadi.dir/generic_type.cpp.o
[ 44%] Building CXX object casadi/core/CMakeFiles/casadi.dir/options_functionality.cpp.o
Linking CXX executable ../bin/profilereport
[ 44%] Built target profilereport
[ 44%] Building CXX object casadi/core/CMakeFiles/casadi.dir/std_vector_tools.cpp.o
[ 46%] Building CXX object casadi/core/CMakeFiles/casadi.dir/functor.cpp.o
[ 46%] Building CXX object casadi/core/CMakeFiles/casadi.dir/functor_internal.cpp.o
[ 46%] Building CXX object casadi/core/CMakeFiles/casadi.dir/polynomial.cpp.o
[ 46%] Building CXX object casadi/core/CMakeFiles/casadi.dir/matrix/matrix.cpp.o
[ 46%] Building CXX object casadi/core/CMakeFiles/casadi.dir/matrix/sparsity.cpp.o
[ 47%] Building CXX object casadi/core/CMakeFiles/casadi.dir/matrix/sparsity_internal.cpp.o
[ 47%] Building CXX object casadi/core/CMakeFiles/casadi.dir/matrix/slice.cpp.o
[ 47%] Building CXX object casadi/core/CMakeFiles/casadi.dir/matrix/matrix_tools.cpp.o
[ 47%] Building CXX object casadi/core/CMakeFiles/casadi.dir/sx/sx_element.cpp.o
[ 47%] Building CXX object casadi/core/CMakeFiles/casadi.dir/sx/sx_node.cpp.o
[ 47%] Building CXX object casadi/core/CMakeFiles/casadi.dir/sx/sx_tools.cpp.o
[ 48%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/mx.cpp.o
[ 48%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/mx_node.cpp.o
[ 48%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/constant_mx.cpp.o
[ 48%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/symbolic_mx.cpp.o
[ 48%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/unary_mx.cpp.o
[ 50%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/multiplication.cpp.o
[ 50%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/casadi_call.cpp.o
[ 50%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/casadi_switch.cpp.o
[ 50%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/casadi_find.cpp.o
[ 50%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/casadi_map.cpp.o
[ 51%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/norm.cpp.o
[ 51%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/transpose.cpp.o
[ 51%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/concat.cpp.o
[ 51%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/split.cpp.o
[ 51%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/reshape.cpp.o
[ 52%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/subref.cpp.o
[ 52%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/subassign.cpp.o
[ 52%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/getnonzeros.cpp.o
[ 52%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/set_sparse.cpp.o
[ 52%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/determinant.cpp.o
[ 53%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/inverse.cpp.o
[ 53%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/inner_prod.cpp.o
[ 53%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/multiple_output.cpp.o
[ 53%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/assertion.cpp.o
[ 53%] Building CXX object casadi/core/CMakeFiles/casadi.dir/mx/monitor.cpp.o
[ 53%] Building CXX object casadi/core/CMakeFiles/casadi.dir/function/io_scheme.cpp.o
[ 55%] Building CXX object casadi/core/CMakeFiles/casadi.dir/function/io_scheme_internal.cpp.o
[ 55%] Building CXX object casadi/core/CMakeFiles/casadi.dir/function/function.cpp.o
[ 55%] Building CXX object casadi/core/CMakeFiles/casadi.dir/function/function_internal.cpp.o
[ 55%] Building CXX object casadi/core/CMakeFiles/casadi.dir/function/sx_function.cpp.o
[ 55%] Building CXX object casadi/core/CMakeFiles/casadi.dir/function/sx_function_internal.cpp.o
[ 56%] Building CXX object casadi/core/CMakeFiles/casadi.dir/function/mx_function.cpp.o
[ 56%] Building CXX object casadi/core/CMakeFiles/casadi.dir/function/mx_function_internal.cpp.o
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/function/mx_function_internal.cpp: In member function ‘virtual void casadi::MXFunctionInternal::generateBody(casadi::CodeGenerator&) const’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/function/mx_function_internal.cpp:1193:10: error: unused variable ‘codegen_class’ [-Werror=unused-variable]
     bool codegen_class = true;
          ^
cc1plus: all warnings being treated as errors
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/function/mx_function_internal.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [casadi/core/CMakeFiles/casadi.dir/all] Error 2
make: *** [all] Error 2

```


any idea why python needs *args? I got this problem before.


I wonder what if you call Money.new(attr).anything ? That method doesn't exist and method_missing will throw an exception while parsing code. Shouldn't we handle it ?


@dwainm I'm thinking the teacher metabox should probably only be visible to admins?

Teachers shouldn't really be able to assign courses to other teachers, right? Any thoughts?


0.8.0 will contain this change, and it may be released in this week or in next week.


Silently switching to a smaller number will give lots of headaches when trying to find out why the i2c transactions are slower than it should.  I just checked in ubuntu 12.04, 14.04 and Archlinux. All of them have the I2C_RDRW_IOCTL_MAX_MSGS definition in /usr/include/linux/i2c-dev.h.  Are you cross-compiling and thus using other headers? (Note it also passed on travis, which uses ubuntu)

Looking at the kernel headers, I2C_RDRW_IOCTL_MAX_MSGS definition is exposed to userspace at least since the start of git era:

> ^1da177e (Linus Torvalds 2005-04-16 15:20:36 -0700 69) #define  I2C_RDRW_IOCTL_MAX_MSGS 42


@tmat It is intentional that exceptions are not handled here, as different hosts will need to do different things. VS, for example, will need to catch the exceptions, warn the user that there are issues with their analyzers, and keep going. VBCSCompiler.exe, however, will let the exception abort the submission and fall back to csc.exe/vbc.exe.

I will, however, go back and document that all of these methods can throw.


@danjjohnson thought about that one and thought the teachers would appreciate this power specially within a market place passing courses / selling it to one another. Also if it is in a school setup and currently there is no teacher assistant you can pass your course to some one for marking etc.


OK


Yes, it is expected. It's a CDN. We fallback on the WebJar when the CDN is not available.


I would wait for someone else to confirm this change `Object -> Record`. Besides this, this pr is good to me


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


We need to restrict the `cmd` key only on OS X here since we reuse the same `EF_COMMAND_DOWN` on Windows/Linux in `super` key.


@tmat OK, I'll change that.


@tmat: That sounds reasonable.


Fair enough. Will add.


en esta linea hay un error


@tmat: That's reasonable.


C'est normal ces liens en dur ?


Use ternary operator :)


Das nächste Mal bitte erst updaten, bevor du weiterarbeitest. Mein mühseliger Kommentar ist jetzt verloren gegangen :-(


The intention was to remove `new Horde_Imap_Client_Mailbox()` from most use case where trash box already exists, it's only the first time when they use it we will need to create a mailbox, so we could have just used the one from getSpecialFolder() where it was already constructed.

But of course we could do it while keeping track of getFolderId(), I do it this way only because this is the only place where we need trash box ID so I don't see the reason of assigning it. I could still do it if you prefer.


@nchase I see.
I'm asking this because [Grails UiPerformance Plugin](https://grails.org/plugin/ui-performance)'s minifier stripped out the function, so `var Toolbar` became `undefined`.


@w0rm @nDmitry @jeffjewiss can you update `CssSyntaxError` error check in your libraries. This one is cleaner and safer.


sorry, bad git push operation,  only want to commit pkg/proxy/router/multioperator.go for a pull request:(
@c4pt0r  please make a rollback, sorry


It would be neater still to have NAN_DISALLOW_ASSIGN and NAN_DISALLOW_COPY mark the functions as deleted in C++11 mode.  That way, you get an error at compile time instead of link or load time.


top level, please


Maybe indent for readability?


Irgendwie verpufft der Donner. Ist das so gewollt?


This is a demo!


It doesn't need args, I just did that because I'm lazy.  The context manager exit function (`__exit__`) takes information about any exceptions that may have been raised inside the context manager, specifically so the exit-er can deal with that information (you may want to do different cleanup depending on whether or not an exception was raised).

So the exception type, exception instance, and traceback objects are all passed to `__exit__` (they might be `None` if no exception was raised).  So I could have done this instead:

```Python
    def __exit__(self, exctype, excinst, tb):
        # Do whatever you need to
```


why style.display none?


Hi,

This was to test lightadmin with an existing project. I could compile and
use it correctly (to the extent of that test), so yes, but it could have
remaining pieces I didn't see.
El dia 05/05/2015 4:32 p. m., "Lazaridis" <notifications@github.com> va
escriure:

> @bfcapell <https://github.com/bfcapell>, were you successfull with all
> those changes?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bfcapell/light-admin/commit/23d3a18e4404bc4ac986ba54d653cd68f99e664a#commitcomment-11045708>
> .
>


I haven't given this very thorough testing, but the new code looks much better than the old one. I have tested it however.


Done add85afed5944ec10d68ca10e91421e031fe0a5d, 17ce9edf1e60f14983b4ff970c9511689c876d5c


aaaa


I will merge it if you adding new option that enable label and text label


how can i  implement it on love2d?


@bpisoj Did you see that Travis is failing?


NICE!


Sweet, thanks... I ask because I was branching to do this change myself :dancers:


I like deleting code! Thanks for this


> Hm, I don't think we should copy random headers around.

OK, we can directly check if the symbol "SetCurrentConsoleFontEx" is defined in the DLL

> Mixxx should build with an off-the-shelf Windows SDK install.

YES!. The current build log proves, that we do not have this state. "v120_xp" does not have VersionHelpers.h since the compiler finds it. We are not an a plain v120_xp SDK.

> Which crash? And what would console font changes have to do with a memory leak?

https://bugs.launchpad.net/mixxx/+bug/1448630
The console output has  hopefully nothing to do with it. But probably a messed up SDK.

> Also, why is any console code running at all in non DEBUGCONSOLE mode?

Every Qt Application is a console Application by default, but it is broken on windows
https://bugreports.qt.io/browse/QTBUG-13303
Fixing this is independent form a "DEBUGCONSOLE" flag since it is possible to launch Mixxx from a console.

For me the "DEBUGCONSOLE" flag including the pop-up console window is an outdated concept anyway. The user has already thee ways to watch the debug output
 1. start in a console
 2. the developer menu
 3. the log file.
For me there is no reason to pop-up a console on windows only. We should just be sure that the default console output works as desired.

Open points:
* figure out how to check if we have already a console window, without accidentally rise it.
* check if we have SetCurrentConsoleFontEx
** Solution a: extract the relevant code form VersionHelpers.h
** Solution b: check for GetCurrentConsoleFontEx
```
HMODULE kernel32_dll = LoadLibraryW(L"kernel32.dll");
if(kernel32_dll &&  GetProcAddress(kernel32_dll, "GetCurrentConsoleFontEx") {
...
```
I wonder if we have to use the address of GetCurrentConsoleFontEx if we are on a clean XP SDK.


Sorry... fixing it...


This is not what these notifications are for. "Finished rendering" means that all tiles which are on screen are fully rendered and the map is complete.


I agree, that is more clearer and indicative of what is happening. I'll make the change.


I'll leave it as ``open`` and correct the commit message typo when I wake up and update the PR (so, 9 hours from now or so.)


I haven't looked into to it but the sticky tests don't pass for me on OS X.


###Milestone 2 Feedback
Excellent job Tim! Your persona and use case are well defined, appropriately detailed, and provide a solid foundation upon which to begin the design and development of your site. Great job identifying your target  user base. Your HTML code looks great. I found only one error, which was a missing alt attribute on the <img> tag. Your Milestone 2 passes at Tier III.

###Suggestions
Remember to include the alt attribute on all images, and don't forget to add a forward-slash (/) to all self-closing tags per the Coding Style Guide. Overall, excellent work!


See https://developer.apple.com/library/ios/documentation/MapKit/Reference/MKMapViewDelegate_Protocol/index.html#//apple_ref/occ/intfm/MKMapViewDelegate/mapViewDidFinishRenderingMap:fullyRendered:


use the brackets so this matches the other statements.


The empty value here should be an empty hash


It should always work, unless the group->order is even for some reason.


This commit has broken the way keystone is integrated with existing node apps. While integrating keystone, it's evident that `keystone.mount()` is never called. As `keystone.static(app)` is no longer available, could you please point the alternate way to configure the same, with this change it's not really clear how to bring that functionality again.

Calling `.mount()` isn't feasible as it tries to reassociate the db object, by opening a new connection while it's already open, which throws an exception.


This property has been updated to point to PROD, as Membership has no code.dev- environment @phamann


Otherwise you are going to get a delegate callback every frame draw.


You have done it thanks.
I think I will revert the current commit at the end, and introduce a better html cleaner that take care of the current output syntax.
Thanks


Yes, I will change this in a new commit. Thanks!


Why exactly? You aren't using stdin later. I suspect that you wanted to go back to interactive after reading the purge list from stdin? This will probably be frowned upon by unix types who may want the ability to pipe things to stdin. If you want this behavior, take the filename as an argument and read from the file instead of stdin.


It will never be true, because string args come in.


Just use 2.0.x


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


First page


Actually, updated now.


added deleteReservationState to avoid code duplication. Further refactoring for orchestrator is planned.


You can decide what *sensible* string format could be converted to a hash of key value pairs.


change message as well


The addition of sudo doesn't belong in this patch.
I think it can actually be set in the ansible configs, so if you want to include this, it needs to be a separate patch and configurable.


This is wrong! :)


So exciting!!!!!


I don't know if I agree with the iteration listener change. I'd prefer to keep them on the layers themselves and perhaps even find a good serialization path. Some cases in ML are common enough (such as score printing every n iterations) that we need this anyways.


It's coming along :)


Removed code duplication.


Please move the conversion to OnTheFlyJavaCompiler2, as GeneratorConfig doesn't hold  anything interesting besides the Java version.


Spelled correctly!


Make up your mind brunch!


Exactly. You can still pipe the file list to stdin. Making the file an input doesn't allow you to pipe filenames into the process. I can imagine a list of urls coming from a curl call or something similar.


Were flags optimizations fault, now system boots. Anyway, can any of you send me your personal kernel build script? @ChronoMonochrome @ace2nutzer


Should we maybe just link to this page: http://www.mono-project.com/docs/about-mono/releases/ ?
Or will this get updated with each release?


fixed


Ping @natefoo.


Kolikor sem prebral o in sprobaval first-class module se mi zdi, da takšna funkcija ne bo mogoča. Problem je, da če narediš seznam Metric modulov, vsi pozabijo, na katerem tipu so definirani.
Npr če je `Abs` modul, ki definira absolutno razdaljo na floatih, potem z `let f = (module Abs : Metric)` dobiš iz njega modul prvega reda. Vendar ko ga odpakiraš z `module NovAbs = (val Abs : Metric)`, funkcija `NewAbs.dist` ne dela več na floatih.
Edina rešitev, ki sem jo našel, je, da se povsod piše `Metric with type point = float`. Potem računanje razdalje deluje, vendar se ne da narediti seznama, ki vsebuje npr `Metric with type point = float` in `Metric with type point = int`. Kar potem nima smisla, ker bi se dalo delati samo potence istega prostora.
Bom še malo pobrskal, če se da to kako drugače rešiti.


This script fails now as it has an extra line at the beginning now. This causes an error:

/mnt/vbox/shared/Arduino_STM32_Sketch/hardware/Arduino_STM32/tools/linux/stlink_upload: 5: /mnt/vbox/shared/Arduino_STM32_Sketch/hardware/Arduino_STM32/tools/linux/stlink_upload: Syntax error: "(" unexpected

http://stackoverflow.com/questions/12910744/why-should-the-shebang-line-always-be-the-first-line


1.1.5#发布文本时在用户中心最低高度为两行的高度。负责相关信息会超出显示范围。


Less maintenance needed = good.


Главное поменять циферку версии :sunglasses:


Damn, that's bad. If #1326 will definitely fix this I agree we should consider merging it now rather than later. The alternatives are to back this out and keep trying to patch up the shared pointer lifetime issues (failed in several tries so far), or address #1364/#926 head-on.


I was very afraid of this chunk of work, great that you've done it. Are you interested in getting those changes back to the original repo? It seems you would just have to change the version back to `1.1.0.BUILD-SNAPSHOT` and then send a PR (there's even a `spring-data-rest-2.3.0` branch on the repo, which was for this upgrade that you've done). If you're to busy, then I'll see to it that your changes go in (in some way).


holy hardcoded paths batman!  You do realize that that's in the COMMON_CMAKE_ARGS already, right? (See the file in the parent path.


If this isn't set by COMMON_CMAKE_ARGS it should be.


Reasonably certain this is handled by common_cmake_args


Yes. Maybe we'll need to hook into it some other way. But we don't even know how to get back to a good state yet.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


These should all get auto-detected by properly building those projects with externalproject_add


Well yes, most command line utilities deal with that by taking "-" as an arg for stdin. I've never used a command line utility that manually switched stdin on me. You can also do <(some command with output) with a filename argument to get the same effect.


I believe you can accept the linky filter as a parameter to your filter function and then use that only on the github urls.


Yep, that does seem like a better solution. Fixed.


I think Chris reported the same.  Can you please retag them and annotate them indicating there's a problem with Mac?  That way I'll know not to untag them in the future.


i think i added it to be a good netizen and save disk space... but peta-bytes are cheap now.


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


The reason you seemed to need this is because you provided the `JSONCPP_LIBRARY`, etc. above instead of having it build and install to the right path.


fixed


This does not look ok to me. Why whould `p.language` be None? Why hardcode _English_ here?


done


cool, thanks


Line 196 has a typo .. 648 instead of 64

+genericSTM32F103C.menu.device_variant.STM32F103C8=STM32F103C8 (20k RAM. 648k Flash)
vs
+genericSTM32F103C.menu.device_variant.STM32F103C8=STM32F103C8 (20k RAM. 64k Flash)


I'll probably revert some of those formatting changes before submitting to gerrit.


I’ve always seen docblocks ending with `*/` as far as I remember. Have you tested that with a tool @stoeffel? https://github.com/tj/dox for example?


Thanks!


:blush:


Looks like you misspelled "canola"


Revenue grade meter is defined by ANSI C12.20


thanks!